Changeset 528 for trunk/client/src/smbwrp.c

Timestamp:

Jan 24, 2011, 12:04:13 PM (15 years ago)

Author:

Silvan Scherrer

Message:

Samba Client 2.1: more errno fixes

File:

• trunk/client/src/smbwrp.c (modified) (5 diffs)

trunk/client/src/smbwrp.c

@@ -174 +174 @@
 
         if (!lp_load(get_dyn_CONFIGFILE(),true,false,false,true)) {
-                debuglocal(0,("The initial smb.conf is missing, please reinstall the plugin!"));
-        }
-
+                debuglocal(0,("The initial smb.conf is missing, defaults are used!\n"));
+        }
 
         load_interfaces();
@@ -959 +958 @@
         if (cli->capabilities & CAP_NT_SMBS && !(cli->capabilities & CAP_NOPATHINFO2)) 
         {
-                int rc = os2cli_errno(cli);
+                int ret = cli_errno(cli);
                 // cli_qpathinfo* reports EINVAL when path of given file not exists
                 // thus there is no real situation when EINVAL should be returned to 
                 // client at this point, we just replace it to ENOTDIR
-                if (rc == EINVAL)
-                {
-                        rc = maperror(ENOTDIR);
-                }
-                return rc;
+                if (ret == EINVAL)
+                {
+                        ret = ENOTDIR;
+                }
+                return maperror(ret);
         }
 
@@ -1031 +1030 @@
         {
                 filelist_state * st  = (filelist_state *)state;
-                char fullname[ _MAX_PATH];
+                char fullname[ _MAX_PATH] = {0};
                 debuglocal(8,"adding <%s> %d %d %d\n", finfo->fname, sizeof(st->finfo), st->datalen, sizeof(st->finfo.fname));
                 memcpy(&st->finfo, finfo, sizeof(st->finfo));
@@ -1099 +1098 @@
 ****************************************************************************/
 // YD from libsmb\clilist.c
-// @todo who frees fname, as clistr_pull_talloc allocs it
 static size_t _os2_interpret_long_filename(TALLOC_CTX *ctx, struct cli_state *cli,
                                    int level,const char *p, const char *pdata_end, smbwrp_fileinfo *finfo,
@@ -1249 +1247 @@
                         }
                         strncpy(finfo->fname, fname, sizeof(finfo->fname) -1);
+
                         /* To be robust in the face of unicode conversion failures
                            we need to copy the raw bytes of the last name seen here.