Page MenuHomeFreeBSD

Do not double free the memory in if_clone.
ClosedPublic

Authored by kan on Apr 13 2017, 4:32 PM.

Details

Summary

if_clone_attach function will drop the reference on failure which will
free the if_clone structure. No need to do it second time.

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

kan created this revision.Apr 13 2017, 4:32 PM
ae accepted this revision.Apr 13 2017, 5:19 PM

I'm not glebius@, but the patch looks correct.

This revision is now accepted and ready to land.Apr 13 2017, 5:19 PM
glebius accepted this revision.Apr 25 2017, 4:23 PM

Thanks! Please commit whitespace separately from functional changes.

This revision was automatically updated to reflect the committed changes.