samba-cvs.cvs
[Top] [All Lists]

svn commit: samba r13665 - in trunk/source/libsmb: .

Subject: svn commit: samba r13665 - in trunk/source/libsmb: .
From:
Date: Fri, 24 Feb 2006 05:05:17 +0000 GMT
Author: jra
Date: 2006-02-24 05:05:13 +0000 (Fri, 24 Feb 2006)
New Revision: 13665

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=13665

Log:
Fix the cli_error codes to always detect a socket error.
This code needs a tidyup and common code with libsmb/errormap.c
merging. Should fix the winbindd crash Jerry found (I hope).
Jeremy.

Modified:
   trunk/source/libsmb/clierror.c


Changeset:
Modified: trunk/source/libsmb/clierror.c
===================================================================
--- trunk/source/libsmb/clierror.c      2006-02-24 05:05:09 UTC (rev 13664)
+++ trunk/source/libsmb/clierror.c      2006-02-24 05:05:13 UTC (rev 13665)
@@ -3,6 +3,7 @@
    client error handling routines
    Copyright (C) Andrew Tridgell 1994-1998
    Copyright (C) Jelmer Vernooij 2003
+   Copyright (C) Jeremy Allison 2006
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
@@ -29,49 +30,73 @@
 
 *******************************************************/
 
-static const struct
-{
-  int err;
-  const char *message;
-} rap_errmap[] =
-{
-  {5,    "RAP5: User has insufficient privilege" },
-  {50,   "RAP50: Not supported by server" },
-  {65,   "RAP65: Access denied" },
-  {86,   "RAP86: The specified password is invalid" },
-  {2220, "RAP2220: Group does not exist" },
-  {2221, "RAP2221: User does not exist" },
-  {2226, "RAP2226: Operation only permitted on a Primary Domain Controller"  },
-  {2237, "RAP2237: User is not in group" },
-  {2242, "RAP2242: The password of this user has expired." },
-  {2243, "RAP2243: The password of this user cannot change." },
-  {2244, "RAP2244: This password cannot be used now (password history 
conflict)." },
-  {2245, "RAP2245: The password is shorter than required." },
-  {2246, "RAP2246: The password of this user is too recent to change."},
+static const struct {
+       int err;
+       const char *message;
+} rap_errmap[] = {
+       {5,    "RAP5: User has insufficient privilege" },
+       {50,   "RAP50: Not supported by server" },
+       {65,   "RAP65: Access denied" },
+       {86,   "RAP86: The specified password is invalid" },
+       {2220, "RAP2220: Group does not exist" },
+       {2221, "RAP2221: User does not exist" },
+       {2226, "RAP2226: Operation only permitted on a Primary Domain 
Controller"  },
+       {2237, "RAP2237: User is not in group" },
+       {2242, "RAP2242: The password of this user has expired." },
+       {2243, "RAP2243: The password of this user cannot change." },
+       {2244, "RAP2244: This password cannot be used now (password history 
conflict)." },
+       {2245, "RAP2245: The password is shorter than required." },
+       {2246, "RAP2246: The password of this user is too recent to change."},
 
-  /* these really shouldn't be here ... */
-  {0x80, "Not listening on called name"},
-  {0x81, "Not listening for calling name"},
-  {0x82, "Called name not present"},
-  {0x83, "Called name present, but insufficient resources"},
+       /* these really shouldn't be here ... */
+       {0x80, "Not listening on called name"},
+       {0x81, "Not listening for calling name"},
+       {0x82, "Called name not present"},
+       {0x83, "Called name present, but insufficient resources"},
 
-  {0, NULL}
+       {0, NULL}
 };  
 
 /****************************************************************************
-  return a description of an SMB error
+ Return a description of an SMB error.
 ****************************************************************************/
+
 static const char *cli_smb_errstr(struct cli_state *cli)
 {
        return smb_dos_errstr(cli->inbuf);
 }
 
+/****************************************************************************
+ Convert a socket error into an NTSTATUS.
+****************************************************************************/
+
+static NTSTATUS cli_smb_rw_error_to_ntstatus(struct cli_state *cli)
+{
+       switch(cli->smb_rw_error) {
+               case READ_TIMEOUT:
+                       return NT_STATUS_IO_TIMEOUT;
+               case READ_EOF:
+                       return NT_STATUS_END_OF_FILE;
+               /* What we shoud really do for read/write errors is convert 
from errno. */
+               /* FIXME. JRA. */
+               case READ_ERROR:
+                       return NT_STATUS_INVALID_NETWORK_RESPONSE;
+               case WRITE_ERROR:
+                       return NT_STATUS_UNEXPECTED_NETWORK_ERROR;
+               case READ_BAD_SIG:
+                       return NT_STATUS_INVALID_PARAMETER;
+               default:
+                       break;
+       }
+       return NT_STATUS_UNSUCCESSFUL;
+}
+
 /***************************************************************************
  Return an error message - either an NT error, SMB error or a RAP error.
  Note some of the NT errors are actually warnings or "informational" errors
  in which case they can be safely ignored.
 ****************************************************************************/
-    
+
 const char *cli_errstr(struct cli_state *cli)
 {   
        static fstring cli_error_message;
@@ -145,11 +170,19 @@
 }
 
 
-/* Return the 32-bit NT status code from the last packet */
+/****************************************************************************
+ Return the 32-bit NT status code from the last packet.
+****************************************************************************/
+
 NTSTATUS cli_nt_error(struct cli_state *cli)
 {
         int flgs2 = SVAL(cli->inbuf,smb_flg2);
 
+       /* Deal with socket errors first. */
+       if (cli->fd == -1 && cli->smb_rw_error) {
+               return cli_smb_rw_error_to_ntstatus(cli);
+       }
+
        if (!(flgs2 & FLAGS2_32_BIT_ERROR_CODES)) {
                int e_class  = CVAL(cli->inbuf,smb_rcls);
                int code  = SVAL(cli->inbuf,smb_err);
@@ -160,16 +193,28 @@
 }
 
 
-/* Return the DOS error from the last packet - an error class and an error
-   code. */
+/****************************************************************************
+ Return the DOS error from the last packet - an error class and an error
+ code.
+****************************************************************************/
+
 void cli_dos_error(struct cli_state *cli, uint8 *eclass, uint32 *ecode)
 {
        int  flgs2;
        char rcls;
        int code;
 
-       if(!cli->initialised) return;
+       if(!cli->initialised) {
+               return;
+       }
 
+       /* Deal with socket errors first. */
+       if (cli->fd == -1 && cli->smb_rw_error) {
+               NTSTATUS status = cli_smb_rw_error_to_ntstatus(cli);
+               ntstatus_to_dos( status, eclass, ecode);
+               return;
+       }
+
        flgs2 = SVAL(cli->inbuf,smb_flg2);
 
        if (flgs2 & FLAGS2_32_BIT_ERROR_CODES) {
@@ -185,6 +230,10 @@
        if (ecode) *ecode    = code;
 }
 
+/****************************************************************************
+ The following mappings need tidying up and moving into libsmb/errormap.c...
+****************************************************************************/
+
 /* Return a UNIX errno from a dos error class, error number tuple */
 
 static int cli_errno_from_dos(uint8 eclass, uint32 num)
@@ -327,6 +376,10 @@
        {NT_STATUS(0), 0}
 };
 
+/****************************************************************************
+ The following mappings need tidying up and moving into libsmb/errormap.c...
+****************************************************************************/
+
 static int cli_errno_from_nt(NTSTATUS status)
 {
        int i;
@@ -334,8 +387,9 @@
 
         /* Status codes without this bit set are not errors */
 
-        if (!(NT_STATUS_V(status) & 0xc0000000))
+        if (!(NT_STATUS_V(status) & 0xc0000000)) {
                 return 0;
+       }
 
        for (i=0;nt_errno_map[i].error;i++) {
                if (NT_STATUS_V(nt_errno_map[i].status) ==
@@ -351,7 +405,10 @@
 
 int cli_errno(struct cli_state *cli)
 {
-        NTSTATUS status;
+       if (cli_is_nt_error(cli)) {
+               NTSTATUS status = cli_nt_error(cli);
+               return cli_errno_from_nt(status);
+       }
 
         if (cli_is_dos_error(cli)) {
                 uint8 eclass;
@@ -361,9 +418,8 @@
                 return cli_errno_from_dos(eclass, ecode);
         }
 
-        status = cli_nt_error(cli);
-
-        return cli_errno_from_nt(status);
+       /* for other cases */
+       return EINVAL;
 }
 
 /* Return true if the last packet was in error */
@@ -372,8 +428,10 @@
 {
        uint32 flgs2 = SVAL(cli->inbuf,smb_flg2), rcls = 0;
 
-       if (cli->fd == -1 && cli->smb_rw_error != 0)
+       /* A socket error is always an error. */
+       if (cli->fd == -1 && cli->smb_rw_error != 0) {
                return True;
+       }
 
         if (flgs2 & FLAGS2_32_BIT_ERROR_CODES) {
                 /* Return error is error bits are set */
@@ -393,6 +451,11 @@
 {
        uint32 flgs2 = SVAL(cli->inbuf,smb_flg2);
 
+       /* A socket error is always an NT error. */
+       if (cli->fd == -1 && cli->smb_rw_error != 0) {
+               return True;
+       }
+
         return cli_is_error(cli) && (flgs2 & FLAGS2_32_BIT_ERROR_CODES);
 }
 
@@ -402,6 +465,11 @@
 {
        uint32 flgs2 = SVAL(cli->inbuf,smb_flg2);
 
+       /* A socket error is always a DOS error. */
+       if (cli->fd == -1 && cli->smb_rw_error != 0) {
+               return True;
+       }
+
         return cli_is_error(cli) && !(flgs2 & FLAGS2_32_BIT_ERROR_CODES);
 }
 

<Prev in Thread] Current Thread [Next in Thread>
  • svn commit: samba r13665 - in trunk/source/libsmb: ., jra <=