diff options
author | Jeremy Allison <jra@samba.org> | 2003-10-02 17:26:44 +0000 |
---|---|---|
committer | Jeremy Allison <jra@samba.org> | 2003-10-02 17:26:44 +0000 |
commit | 64f88c008cc4f047ecd8e457fbdcafde32e9def5 (patch) | |
tree | a7ab134165f85e688f7c8f2d89808d1f7000d318 | |
parent | 418749050ba387a4657cc23dfb86cf06548703cf (diff) | |
download | samba-64f88c008cc4f047ecd8e457fbdcafde32e9def5.tar.gz samba-64f88c008cc4f047ecd8e457fbdcafde32e9def5.tar.bz2 samba-64f88c008cc4f047ecd8e457fbdcafde32e9def5.zip |
Correct fix for excel read-only bug. Add panic for logic error in developer mode.
Jeremy.
(This used to be commit 0857932c1c1960fd68a10f3d1f8c300b7a28b6d7)
-rw-r--r-- | source3/smbd/error.c | 10 | ||||
-rw-r--r-- | source3/smbd/trans2.c | 3 |
2 files changed, 7 insertions, 6 deletions
diff --git a/source3/smbd/error.c b/source3/smbd/error.c index 7eec5e25df..795bf0949c 100644 --- a/source3/smbd/error.c +++ b/source3/smbd/error.c @@ -61,6 +61,9 @@ int unix_error_packet(char *outbuf,int def_class,uint32 def_code, eclass = unix_ERR_class; ecode = unix_ERR_code; ntstatus = unix_ERR_ntstatus; + unix_ERR_class = SMB_SUCCESS; + unix_ERR_code = 0; + unix_ERR_ntstatus = NT_STATUS_OK; } else { while (unix_dos_nt_errmap[i].dos_class != 0) { if (unix_dos_nt_errmap[i].unix_error == errno) { @@ -90,9 +93,10 @@ int error_packet(char *outbuf,NTSTATUS ntstatus, if (errno != 0) DEBUG(3,("error string = %s\n",strerror(errno))); - unix_ERR_class = SMB_SUCCESS; - unix_ERR_code = 0; - unix_ERR_ntstatus = NT_STATUS_OK; +#if defined(DEVELOPER) + if (unix_ERR_class != SMB_SUCCESS || unix_ERR_code != 0 || !NT_STATUS_IS_OK(unix_ERR_ntstatus)) + smb_panic("logic error in error processing"); +#endif /* * We can explicitly force 32 bit error codes even when the diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c index 033e76a33e..ee63220d18 100644 --- a/source3/smbd/trans2.c +++ b/source3/smbd/trans2.c @@ -1785,12 +1785,9 @@ int set_bad_path_error(int err, BOOL bad_path, char *outbuf, int def_class, uint err, (int)bad_path )); if(err == ENOENT) { - unix_ERR_class = ERRDOS; if (bad_path) { - unix_ERR_code = ERRbadpath; return ERROR_NT(NT_STATUS_OBJECT_PATH_NOT_FOUND); } else { - unix_ERR_code = ERRbadfile; return ERROR_NT(NT_STATUS_OBJECT_NAME_NOT_FOUND); } } |