Hi tech,

this diff removes an unneeded variable from signify.
The int is only used for a comparison in order to make sure the b64 encoding 
was right.

Regards,
Fritjof

Index: signify.c
===================================================================
RCS file: /cvs/src/usr.bin/signify/signify.c,v
retrieving revision 1.41
diff -u -p -u -r1.41 signify.c
--- signify.c   22 Jan 2014 21:11:03 -0000      1.41
+++ signify.c   27 Feb 2014 13:17:16 -0000
@@ -136,7 +136,6 @@ static size_t
 parseb64file(const char *filename, char *b64, void *buf, size_t len,
     char *comment)
 {
-       int rv;
        char *commentend, *b64end;
 
        commentend = strchr(b64, '\n');
@@ -154,8 +153,7 @@ parseb64file(const char *filename, char 
        if (!b64end)
                errx(1, "missing new line after b64 in %s", filename);
        *b64end = 0;
-       rv = b64_pton(commentend + 1, buf, len);
-       if (rv != len)
+       if (b64_pton(commentend + 1, buf, len) != len)
                errx(1, "invalid b64 encoding in %s", filename);
        if (memcmp(buf, PKALG, 2))
                errx(1, "unsupported file %s", filename);

Reply via email to