diff options
| author | bdrewery <bdrewery@FreeBSD.org> | 2013-11-11 21:42:31 +0000 |
|---|---|---|
| committer | bdrewery <bdrewery@FreeBSD.org> | 2013-11-11 21:42:31 +0000 |
| commit | f0bfa091d03ddd5582a9867f1cd13d331a105b14 (patch) | |
| tree | 5d7bfeb79012e2d46950d60ed6e811c3d4973477 /usr.sbin/pkg | |
| parent | a76f9dffc594021dc974ac898a766e52da9a2f7b (diff) | |
| download | FreeBSD-src-f0bfa091d03ddd5582a9867f1cd13d331a105b14.zip FreeBSD-src-f0bfa091d03ddd5582a9867f1cd13d331a105b14.tar.gz | |
Remove useless checks for NULL pointer before free(3)
Reported by: eadler
Approved by: eadler
Diffstat (limited to 'usr.sbin/pkg')
| -rw-r--r-- | usr.sbin/pkg/pkg.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/usr.sbin/pkg/pkg.c b/usr.sbin/pkg/pkg.c index 76368ac..b4dcfc8 100644 --- a/usr.sbin/pkg/pkg.c +++ b/usr.sbin/pkg/pkg.c @@ -326,8 +326,7 @@ free_fingerprint_list(struct fingerprint_list* list) struct fingerprint *fingerprint, *tmp; STAILQ_FOREACH_SAFE(fingerprint, list, next, tmp) { - if (fingerprint->name) - free(fingerprint->name); + free(fingerprint->name); free(fingerprint); } free(list); @@ -729,12 +728,9 @@ cleanup: if (revoked) free_fingerprint_list(revoked); if (sc) { - if (sc->cert) - free(sc->cert); - if (sc->sig) - free(sc->sig); - if (sc->name) - free(sc->name); + free(sc->cert); + free(sc->sig); + free(sc->name); free(sc); } |
