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
Unknown Object (File)
Jan 26 2026, 2:15 AM
Unknown Object (File)
Jan 22 2026, 5:48 PM
Unknown Object (File)
Jan 20 2026, 1:26 PM
Unknown Object (File)
Jan 10 2026, 6:55 PM
Unknown Object (File)
Nov 18 2025, 9:50 PM
Unknown Object (File)
Oct 11 2025, 10:24 PM
Unknown Object (File)
Oct 11 2025, 10:24 PM
Unknown Object (File)
Oct 11 2025, 1:09 PM
View All 60 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...