Btrfs-progs: fix several complie warning

PATCH New Miao Xie 1 participant 1 message Aug 10, 2012 Btrfs file system

Share & tools

Miao Xie

author

2012-08-10 08:54 UTC #0 lore raw

PATCH v2 1/3 Btrfs-progs: fix several complie warning

This patch fixed the following warning:
cmds-send.c:464:6: warning: ‘ret' may be used uninitialized in this function [-Wuninitialized] crc32c.c:121:1: warning: control reaches end of non-void function [-Wreturn-type]
send-utils.c:69:11: warning: ‘comp' may be used uninitialized in this function [-Wuninitialized] send-utils.c:126:6: warning: ‘comp' may be used uninitialized in this function [-Wuninitialized] send-utils.c:99:22: warning: ‘entry' may be used uninitialized in this function [-Wuninitialized] btrfs.c:261:2: warning: implicit declaration of function ‘crc32c_optimization_init' [-Wimplicit-function-declaration] btrfs.c:105:2: warning: ‘cmd' may be used uninitialized in this function [-Wuninitialized] restore.c:435:12: warning: ignoring return value of ‘ftruncate', declared with attribute warn_unused_result [-Wunused-result]

Signed-off-by Miao Xie <[email protected]>

---
Changelog v1 -> v2:
- just change the title of this patch.

---
 btrfs.c      |    3 ++-
 cmds-send.c  |    2 ++
 crc32c.c     |    4 ++--
 crc32c.h     |    2 +-
 restore.c    |    8 +++++---
 send-utils.c |    4 ++++
 6 files changed, 16 insertions(+), 7 deletions(-)

btrfs.c +2 -1

index e9d54f8..ed99994 100644
@@ -19,6 +19,7 @@
 #include <stdlib.h>
 #include <string.h>
 
+#include "crc32c.h"
 #include "commands.h"
 #include "version.h"

@@ -93,7 +94,7 @@ static int parse_one_token(const char *arg, const struct cmd_group *grp,
 static const struct cmd_struct *
 parse_command_token(const char *arg, const struct cmd_group *grp)
 {
-	const struct cmd_struct *cmd;
+	const struct cmd_struct *cmd = NULL;
 
 	switch(parse_one_token(arg, grp, &cmd)) {
 	case -1:

cmds-send.c +2 -0

index 8a0c110..3d15d26 100644
@@ -541,6 +541,7 @@ int cmd_send_start(int argc, char **argv)
 
 	subvol = realpath(argv[optind], NULL);
 	if (!subvol) {
+		ret = -errno;
 		fprintf(stderr, "ERROR: unable to resolve %s\n", argv[optind]);
 		goto out;
 	}

@@ -565,6 +566,7 @@ int cmd_send_start(int argc, char **argv)
 	for (i = optind; i < argc; i++) {
 		subvol = realpath(argv[i], NULL);
 		if (!subvol) {
+			ret = -errno;
 			fprintf(stderr, "ERROR: unable to resolve %s\n", argv[i]);
 			goto out;
 		}

crc32c.c +2 -2

index a761a1d..c285d6e 100644
@@ -113,7 +113,7 @@ void crc32c_intel_probe(void)
 	}
 }
 
-int crc32c_optimization_init(void)
+void crc32c_optimization_init(void)
 {
 	crc32c_intel_probe();
 	if (crc32c_intel_available)

@@ -121,7 +121,7 @@ int crc32c_optimization_init(void)
 }
 #else
 
-int crc32c_optimization_init(void)
+void crc32c_optimization_init(void)
 {
 }

crc32c.h +1 -1

index 27f298a..7f12e77 100644
@@ -22,7 +22,7 @@
 #include "kerncompat.h"
 
 u32 crc32c_le(u32 seed, unsigned char const *data, size_t length);
-int crc32c_optimization_init(void);
+void crc32c_optimization_init(void);
 
 #define crc32c(seed, data, length) crc32c_le(seed, (unsigned char const *)data, length)
 #define btrfs_crc32c crc32c

restore.c +5 -3

index d1ac542..b9e5381 100644
@@ -385,7 +385,6 @@ static int copy_file(struct btrfs_root *root, int fd, struct btrfs_key *key,
 					/* No more leaves to search */
 					btrfs_free_path(path);
 					goto set_size;
-					return 0;
 				}
 				leaf = path->nodes[0];
 			} while (!leaf);

@@ -431,8 +430,11 @@ next:
 
 	btrfs_free_path(path);
 set_size:
-	if (found_size)
-		ftruncate(fd, (loff_t)found_size);
+	if (found_size) {
+		ret = ftruncate(fd, (loff_t)found_size);
+		if (ret)
+			return ret;
+	}
 	return 0;
 }

send-utils.c +4 -0

index a43d47e..af495a0 100644
@@ -62,6 +62,8 @@ static struct rb_node *tree_insert(struct rb_root *root,
 			entry = rb_entry(parent, struct subvol_info,
 					rb_path_node);
 			comp = strcmp(entry->path, si->path);
+		} else {
+			BUG();
 		}
 
 		if (comp < 0)

@@ -120,6 +122,8 @@ static struct subvol_info *tree_search(struct rb_root *root,
 		} else if (type == subvol_search_by_path) {
 			entry = rb_entry(n, struct subvol_info, rb_path_node);
 			comp = strcmp(entry->path, path);
+		} else {
+			BUG();
 		}
 		if (comp < 0)
 			n = n->rb_left;

Signature · 1 line

1.7.6.5