In gcc version 4.3.2 we get warnings for functions declared with
authorJeremy Allison <jeremy@jeremy-desktop.(none)>
Tue, 23 Dec 2008 04:05:45 +0000 (20:05 -0800)
committerJeremy Allison <jeremy@jeremy-desktop.(none)>
Tue, 23 Dec 2008 04:05:45 +0000 (20:05 -0800)
attribute warn_unused_result. Start to fix these.
Jeremy.

source/lib/socket_wrapper/socket_wrapper.c
source/libnet/libnet_join.c

index 33e4b3837029cc16367c0eb1d59363367a486758..cee43c06a50a224adf91f22dd792fdcd8ffd2c9b 100644 (file)
@@ -899,7 +899,10 @@ static int swrap_get_pcap_fd(const char *fname)
                file_hdr.frame_max_len  = SWRAP_FRAME_LENGTH_MAX;
                file_hdr.link_type      = 0x0065; /* 101 RAW IP */
 
-               write(fd, &file_hdr, sizeof(file_hdr));
+               if (write(fd, &file_hdr, sizeof(file_hdr)) != sizeof(file_hdr)) {
+                       close(fd);
+                       return -1;
+               }
                return fd;
        }
 
@@ -1190,7 +1193,12 @@ static void swrap_dump_packet(struct socket_info *si,
 
        fd = swrap_get_pcap_fd(file_name);
        if (fd != -1) {
-               write(fd, packet, packet_len);
+               if (write(fd, packet, packet_len) != packet_len) {
+                       close(fd);
+                       free(packet);
+                       return;
+               }
+               close(fd);
        }
 
        free(packet);
index 8abd317ca6d08c36059c9c2977bf4d0aeb502628..b8d903cc9a2a0cb721ff42dcd478bcf477ab7275 100644 (file)
@@ -1019,8 +1019,7 @@ NTSTATUS libnet_join_ok(const char *netbios_domain_name,
                return NT_STATUS_NO_TRUST_LSA_SECRET;
        }
 
-       asprintf(&machine_account, "%s$", machine_name);
-       if (!machine_account) {
+       if (asprintf(&machine_account, "%s$", machine_name) == -1) {
                SAFE_FREE(machine_password);
                return NT_STATUS_NO_MEMORY;
        }