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);