Page MenuHomeFreeBSD

uefisign.c: Plug memory leaks.
AcceptedPublic

Authored by delphij on Jun 28 2017, 4:55 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 13, 1:32 PM
Unknown Object (File)
Sep 16 2025, 7:58 AM
Unknown Object (File)
Aug 19 2025, 8:00 AM
Unknown Object (File)
Aug 10 2025, 5:33 AM
Unknown Object (File)
Jul 28 2025, 12:01 PM
Unknown Object (File)
Jul 28 2025, 11:37 AM
Unknown Object (File)
Jul 28 2025, 11:00 AM
Unknown Object (File)
Jul 11 2025, 11:06 PM
Subscribers

Details

Reviewers
trasz
emaste
Summary

Plug memory leaks.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 10162
Build 10585: arc lint + arc unit

Event Timeline

usr.sbin/uefisign/uefisign.c
352–355

early exit would simplify slightly and reduce indentation,

case 'c':
    if (certpath != NULL)
        err(1, "-c can only be specified once");
    certpath = checked_strdup(optarg);
    break;
linimon retitled this revision from Plug memory leaks. to uefisign.c: Plug memory leaks..Jul 19 2018, 5:22 AM
This revision is now accepted and ready to land.Mar 21 2023, 4:12 PM

The exit vs return stuff I don't understand how that really changes anything...

Otherwise I agree with ed.