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 18:49:59 +0000 (19:49 +0100)
(cherry picked from commit aee7d55fcf754d97d945c318ec06d6387ade067b)

daemon/daemon.h
daemon/guestfsd.c
daemon/parted.c

index ebbeaa2..ea30701 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 070684d..0112a7f 100644 (file)
@@ -543,10 +543,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;