* import.c (import_keys_internal): s/inp/inp2/ to avoid shadowing
authorWerner Koch <wk@gnupg.org>
Fri, 4 Oct 2002 05:43:54 +0000 (05:43 +0000)
committerWerner Koch <wk@gnupg.org>
Fri, 4 Oct 2002 05:43:54 +0000 (05:43 +0000)
warning.

* passphrase.c (agent_get_passphrase): Fixed signed/unsigned char
problem in %-escaping.  Noted by Ingo Kl�cker.

g10/ChangeLog
g10/import.c
g10/passphrase.c

index 84f05b2..de855dc 100644 (file)
@@ -1,3 +1,11 @@
+2002-10-04  Werner Koch  <wk@gnupg.org>
+
+       * import.c (import_keys_internal): s/inp/inp2/ to avoid shadowing
+       warning.
+
+       * passphrase.c (agent_get_passphrase): Fixed signed/unsigned char
+       problem in %-escaping.  Noted by Ingo Klöcker.
+
 2002-10-03  David Shaw  <dshaw@jabberwocky.com>
 
        * options.h, g10.c (main): Add --strict and --no-strict to switch
index 9f6b893..30ac913 100644 (file)
@@ -193,14 +193,14 @@ import_keys_internal( IOBUF inp, char **fnames, int nnames,
 
        for(i=0; i < nnames; i++ ) {
            const char *fname = fnames? fnames[i] : NULL;
-           IOBUF inp = iobuf_open(fname);
+           IOBUF inp2 = iobuf_open(fname);
            if( !fname )
                fname = "[stdin]";
            if( !inp )
                log_error(_("can't open `%s': %s\n"), fname, strerror(errno) );
            else {
-               rc = import( inp, fname, stats, options );
-               iobuf_close(inp);
+               rc = import( inp2, fname, stats, options );
+               iobuf_close(inp2);
                if( rc )
                    log_error("import from `%s' failed: %s\n", fname,
                                      g10_errstr(rc) );
index 0da40e3..d84d6e8 100644 (file)
@@ -732,6 +732,7 @@ agent_get_passphrase ( u32 *keyid, int mode, const char *tryagain_text )
   else
     { /* The new Assuan protocol */
       char *line, *p;
+      const unsigned char *s;
       int i; 
 
       if (!tryagain_text)
@@ -751,33 +752,33 @@ agent_get_passphrase ( u32 *keyid, int mode, const char *tryagain_text )
       else
         *p++ = 'X'; /* no caching */
       *p++ = ' ';
-      for (i=0; tryagain_text[i]; i++)
+      for (i=0, s=tryagain_text; *s; s++)
         {
-          if (tryagain_text[i] < ' ' || tryagain_text[i] == '+')
+          if (*s < ' ' || *s == '+')
             {
-              sprintf (p, "%%%02X", tryagain_text[i]);
+              sprintf (p, "%%%02X", *s);
               p += 3;
             }
-          else if (tryagain_text[i] == ' ')
+          else if (*s == ' ')
             *p++ = '+';
           else
-            *p++ = tryagain_text[i];
+            *p++ = *s;
         }
       *p++ = ' ';
       *p++ = 'X'; /* Use the standard prompt */
       *p++ = ' ';
       /* copy description */
-      for (i=0; atext[i]; i++)
+      for (i=0, s= atext; *s; s++)
         {
-          if (atext[i] < ' ' || atext[i] == '+')
+          if (*s < ' ' || *s == '+')
             {
-              sprintf (p, "%%%02X", atext[i]);
+              sprintf (p, "%%%02X", *s);
               p += 3;
             }
-          else if (atext[i] == ' ')
+          else if (*s == ' ')
             *p++ = '+';
           else
-            *p++ = atext[i];
+            *p++ = *s;
         }
       *p++ = '\n';
       i = writen (fd, line, p - line);