Hi,

Here a patch that corrects three err() to errx() calls.
  - a if condition don't set errno
  - strlcpy(3) don't set errno (no mention is man page)
  - ca_readpass() already manage errno error message with warn(3)

Comments ? OKs ?
-- 
Sebastien Marie


Index: ikeca.c
===================================================================
RCS file: /cvs/src/usr.sbin/ikectl/ikeca.c,v
retrieving revision 1.30
diff -u -p -r1.30 ikeca.c
--- ikeca.c     16 Jan 2015 06:40:17 -0000      1.30
+++ ikeca.c     12 Aug 2015 06:43:24 -0000
@@ -219,7 +219,7 @@ ca_sign(struct ca *ca, char *keyname, in
                    ca->sslpath, keyname, ca->sslpath, ca->sslpath,
                    ca->extcnf, ca->sslpath, keyname, ca->passfile);
        } else
-               err(1, "unknown host type %d", type);
+               errx(1, "unknown host type %d", type);
 
        system(cmd);
 
@@ -561,7 +561,7 @@ ca_export(struct ca *ca, char *keyname, 
 
        if (keyname != NULL) {
                if (strlcpy(oname, keyname, sizeof(oname)) >= sizeof(oname))
-                       err(1, "name too long");
+                       errx(1, "name too long");
        } else {
                strlcpy(oname, "ca", sizeof(oname));
        }
@@ -758,7 +758,7 @@ ca_revoke(struct ca *ca, char *keyname)
        snprintf(path, sizeof(path), "%s/ikeca.passwd", ca->sslpath);
        pass = ca_readpass(path, &len);
        if (pass == NULL)
-               err(1, "could not open passphrase file");
+               errx(1, "could not open passphrase file");
 
        /* create index if it doesn't already exist */
        snprintf(path, sizeof(path), "%s/index.txt", ca->sslpath);

Reply via email to