]> granicus.if.org Git - neomutt/commitdiff
fix signed/unsigned comparison in longest_common_prefix
authorBrendan Cully <brendan@kublai.com>
Sat, 8 Jul 2017 23:35:08 +0000 (16:35 -0700)
committerRichard Russon <rich@flatcap.org>
Fri, 14 Jul 2017 11:07:33 +0000 (12:07 +0100)
imap/imap.c

index 746bdc4dfe2fcb0b1bc377b329f515fbed129ec5..bedd09867e7758cd62e7c64648bcacccd24eb91d 100644 (file)
@@ -1965,9 +1965,9 @@ fail:
 
 /* trim dest to the length of the longest prefix it shares with src,
  * returning the length of the trimmed string */
-static int longest_common_prefix(char *dest, const char *src, int start, size_t dlen)
+static size_t longest_common_prefix(char *dest, const char *src, size_t start, size_t dlen)
 {
-  int pos = start;
+  size_t pos = start;
 
   while (pos < dlen && dest[pos] && dest[pos] == src[pos])
     pos++;
@@ -1983,7 +1983,7 @@ static int imap_complete_hosts(char *dest, size_t len)
   struct Buffy *mailbox = NULL;
   struct Connection *conn = NULL;
   int rc = -1;
-  int matchlen;
+  size_t matchlen;
 
   matchlen = mutt_strlen(dest);
   for (mailbox = Incoming; mailbox; mailbox = mailbox->next)
@@ -2037,7 +2037,8 @@ int imap_complete(char *dest, size_t dlen, char *path)
   char buf[LONG_STRING];
   struct ImapList listresp;
   char completion[LONG_STRING];
-  int clen, matchlen = 0;
+  int clen;
+  size_t matchlen = 0;
   int completions = 0;
   struct ImapMbox mx;
   int rc;