verifyDSASignature() buffer & other fixes
authorPanu Matilainen <pmatilai@redhat.com>
Tue, 8 Apr 2008 10:27:59 +0000 (13:27 +0300)
committerPanu Matilainen <pmatilai@redhat.com>
Tue, 8 Apr 2008 10:27:59 +0000 (13:27 +0300)
- avoid stpcpy() into static sized buffer
- avoid potential sigp NULL-dereference

lib/signature.c

index c531e26..cf7aaf0 100644 (file)
@@ -1245,6 +1245,9 @@ verifyDSASignature(rpmts ts, char ** msg,
     const char *hdr;
     int sigver;
 
+    assert(msg != NULL);
+    *msg = NULL;
+
     hdr = (dig != NULL && dig->hdrsha1ctx == sha1ctx) ? _("Header ") : "";
     sigver = sigp !=NULL ? sigp->version : 0;
 
@@ -1314,7 +1317,6 @@ exit:
        rasprintf(msg, _("%sV%d DSA signature: %s\n"),
                  hdr, sigver, rpmSigString(res));
     }
-
     return res;
 }