daemon: count_strings function returns size_t
authorRichard Jones <rjones@redhat.com>
Wed, 2 Jun 2010 11:31:40 +0000 (12:31 +0100)
committerRichard Jones <rjones@redhat.com>
Wed, 2 Jun 2010 14:33:02 +0000 (15:33 +0100)
daemon/daemon.h
daemon/guestfsd.c
daemon/parted.c

index de598cd..d90b65c 100644 (file)
@@ -43,7 +43,7 @@ extern int xread (int sock, void *buf, size_t len)
   __attribute__((__warn_unused_result__));
 
 extern int add_string (char ***argv, int *size, int *alloc, const char *str);
-extern int count_strings (char *const *argv);
+extern size_t count_strings (char *const *argv);
 extern void sort_strings (char **argv, int len);
 extern void free_strings (char **argv);
 extern void free_stringslen (char **argv, int len);
index c0d524a..f9e5a68 100644 (file)
@@ -544,10 +544,10 @@ add_string (char ***argv, int *size, int *alloc, const char *str)
   return 0;
 }
 
-int
+size_t
 count_strings (char *const *argv)
 {
-  int argc;
+  size_t argc;
 
   for (argc = 0; argv[argc] != NULL; ++argc)
     ;
index 9c65570..bf45f8b 100644 (file)
@@ -392,7 +392,7 @@ do_part_get_bootable (const char *device, int partnum)
     return -1;
 
   /* We want lines[1+partnum]. */
-  if (count_strings (lines) < 1+partnum) {
+  if (count_strings (lines) < (size_t) (1+partnum)) {
     reply_with_error ("partition number out of range: %d", partnum);
     free_strings (lines);
     return -1;