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

Timestamp:

Jan 10, 2011, 1:45:04 PM (15 years ago)

Author:

Silvan Scherrer

Message:

Samba Client 2.1: sigpipe crash, cosmetic and text

File:

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

trunk/client/src/smbwrp.c

@@ -195 +195 @@
         }
 */
+        /*
+         * Block SIGPIPE (from lib/util_sock.c: write())
+         * It is not needed and should not stop execution
+         */
+        BlockSignals(True, SIGPIPE);
+
         return 0;
 
@@ -597 +603 @@
         if (!cli || !file || !*file->fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         if (file->denymode < DENY_ALL || file->denymode > DENY_NONE) 
@@ -634 +640 @@
         if (!cli || !file || !buf || !result) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         
@@ -660 +666 @@
         if (!cli || !file || !buf || !result) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         
@@ -684 +690 @@
         if (!cli || !file) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -755 +761 @@
         if (!cli || !file) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -791 +797 @@
         if (!cli || !oldname || !newname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -812 +818 @@
         if (!cli || !finfo || !*finfo->fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -831 +837 @@
         if (!cli || !fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 #if 0
@@ -862 +868 @@
         if (!cli || !file) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -871 +877 @@
                 if (offset < 0)
                 {
-                        return EINVAL;
+                        return maperror(EINVAL);
                 }
                 file->offset = offset;
@@ -881 +887 @@
                 if (offset > 0)
                 {
-                        return EINVAL;
+                        return maperror(EINVAL);
                 }
                 if (!cli_qfileinfo3(cli, file->fd, 
@@ -893 +899 @@
                 file->offset = size + offset;
                 break;
-        default: return EINVAL;
+        default: return maperror(EINVAL);
         }
 
@@ -908 +914 @@
         if (!cli || !finfo || !*finfo->fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         debuglocal(4,"getattr %d %d <%s>\n", cli->capabilities & CAP_NOPATHINFO2, cli->capabilities & CAP_NT_SMBS, finfo->fname);
@@ -954 +960 @@
                 if (rc == EINVAL)
                 {
-                        rc = ENOTDIR;
+                        rc = maperror(ENOTDIR);
                 }
                 return rc;
@@ -980 +986 @@
         if (!cli || !file || !finfo) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1558 +1564 @@
         if (!srv || !cli || !state || !*state->mask)
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         debuglocal(1,"Filelist <%s> on master <%s> wgrp <%s> server <%s> share <%s> clidev <%s>\n", state->mask, srv->master, srv->workgroup, srv->server_name, srv->share_name, cli->dev);
@@ -1628 +1634 @@
         if (!cli || !fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1638 +1644 @@
 
         if (!(finfo.attr & aDIR)) {
-                return ENOTDIR;
+                return maperror(ENOTDIR);
         }
 
@@ -1652 +1658 @@
         if (!cli || !fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1669 +1675 @@
         if (!cli || !fname) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1686 +1692 @@
         if (!cli || !fname || !name) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         if (!cli_set_ea_path(cli, fname, name, value, size)) 
@@ -1702 +1708 @@
         if (!cli || !file || !name) 
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
         if (!cli_set_ea_fnum(cli, file->fd, name, value, size)) 
@@ -1801 +1807 @@
         if (!cli || !fname || !buffer)
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1815 +1821 @@
         if (!cli || !file || !buffer)
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }
 
@@ -1831 +1837 @@
         if (!cli || !pfsa)
         {
-                return EINVAL;
+                return maperror(EINVAL);
         }