Page MenuHomeFreeBSD
Differential D29896

libkiconv: address memory leak in a not-found case
ClosedPublic
Actions

Authored by emaste on Apr 21 2021, 5:48 PM.
Tags
None
Referenced Files
F96152767: D29896.id.diff
Mon, Sep 23, 9:18 PM
Unknown Object (File)
Sat, Sep 14, 3:24 PM
Unknown Object (File)
Thu, Sep 12, 4:13 PM
Unknown Object (File)
Wed, Sep 11, 5:25 PM
Unknown Object (File)
Wed, Sep 11, 4:23 PM
Unknown Object (File)
Wed, Sep 11, 4:23 PM
Unknown Object (File)
Wed, Sep 11, 4:23 PM
Unknown Object (File)
Sat, Sep 7, 5:37 PM
View All 64 Files
Subscribers
jhb
scottl

Details

Reviewers
scottl
imp
jhb
Commits
rG8e64d97d94f7: libkiconv: address memory leak in not-found cases
rG7452b2e44be6: libkiconv: address memory leak in not-found cases
rGe3bbb369c44b: libkiconv: address memory leak in not-found cases
rG3cfd08c1c740: libkiconv: address memory leak in not-found cases
Summary

Found in "Understanding and Detecting Disordered Error Handling with Precise Function Pairing" by Qiushi Wu et. al.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste requested review of this revision.Apr 21 2021, 5:48 PM
emaste created this revision.
scottl requested changes to this revision.Apr 21 2021, 5:59 PM
scottl added a subscriber: scottl.
scottl added inline comments.
lib/libkiconv/kiconv_sysctl.c
90

Do you mean free(csi) here?

This revision now requires changes to proceed.Apr 21 2021, 5:59 PM
emaste updated this revision to Diff 87887.Apr 21 2021, 6:10 PM

fix from scott

emaste added a comment.Apr 21 2021, 6:11 PM

paper was here: https://www-users.cs.umn.edu/~kjlu/papers/hero.pdf

imp accepted this revision.Apr 21 2021, 6:16 PM
jhb accepted this revision.Apr 21 2021, 6:25 PM
jhb added a subscriber: jhb.

RAII would be nice.

This revision was not accepted when it landed; it landed in state Needs Review.Apr 22 2021, 11:34 PM
Closed by commit rG3cfd08c1c740: libkiconv: address memory leak in not-found cases (authored by emaste). · Explain Why
This revision was automatically updated to reflect the committed changes.
emaste added a commit: rG3cfd08c1c740: libkiconv: address memory leak in not-found cases.
emaste added a commit: rGe3bbb369c44b: libkiconv: address memory leak in not-found cases.Apr 28 2021, 12:11 AM
emaste added a commit: rG7452b2e44be6: libkiconv: address memory leak in not-found cases.
emaste added a commit: rG8e64d97d94f7: libkiconv: address memory leak in not-found cases.

Revision Contents
Changeset List

PathSize
lib/
libkiconv/
2 lines

Diff 88021

View Options

lib/libkiconv/kiconv_sysctl.c

Loading...