Page MenuHomeFreeBSD
Differential D9865

Fix a memory leak in i2c
AbandonedPublic
Actions

Authored by stevek on Mar 2 2017, 2:35 PM.
Tags
None
Referenced Files
F81638861: D9865.diff
Fri, Apr 19, 8:28 AM
Unknown Object (File)
Sun, Apr 7, 4:19 AM
Unknown Object (File)
Feb 15 2024, 6:33 PM
Unknown Object (File)
Dec 20 2023, 1:08 AM
Unknown Object (File)
Oct 25 2023, 9:58 AM
Unknown Object (File)
Jun 21 2023, 7:14 AM
Unknown Object (File)
May 5 2023, 9:25 AM
Unknown Object (File)
Apr 18 2023, 6:24 PM
View All 11 Files
Subscribers
imp

Details

Reviewers
zbb
trix_juniper.net
Summary

When the realloc of buf fails, buf is never freed before returning.
Convert to reallocf so buf is freed.

Test Plan

Run clang's static analyzer, scan-build, before to find the problem, run after to show resolution.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

trix_juniper.net updated this revision to Diff 25887.Mar 2 2017, 2:35 PM
trix_juniper.net retitled this revision from to Fix a memory leak in i2c.
trix_juniper.net updated this object.
trix_juniper.net edited the test plan for this revision. (Show Details)
trix_juniper.net added reviewers: stevek, zbb.
trix_juniper.net set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptMar 2 2017, 2:35 PM
stevek commandeered this revision.Apr 18 2023, 5:06 PM
stevek abandoned this revision.
stevek edited reviewers, added: trix_juniper.net; removed: stevek.

Handled by commit e06874f3f6d1f

Revision Contents
Changeset List

PathSize
usr.sbin/
i2c/
2 lines

Diff 25887

View Options

usr.sbin/i2c/i2c.c

Loading...