Fix file descriptor and memory leaks in libdpv
Needs ReviewPublic
Actions

Authored by dteske on Jun 8 2016, 10:00 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Mar 24, 4:18 AM

	Unknown Object (File)
	Mar 6 2026, 4:12 PM

	Unknown Object (File)
	Feb 28 2026, 1:36 PM

	Unknown Object (File)
	Feb 23 2026, 3:42 AM

	Unknown Object (File)
	Jan 16 2026, 2:15 PM

	Unknown Object (File)
	Jan 16 2026, 9:56 AM

	Unknown Object (File)
	Jan 10 2026, 3:06 PM

	Unknown Object (File)
	Jan 10 2026, 4:13 AM

View All 89 Files

Subscribers

imp

Details

Reviewers

truckman

Summary

Close /dev/tty in tty_maxsize_update() if we open it
in the !isatty(fd) case to avoid leaking the descriptor.
Verify that we were actually able to open /dev/tty.

Don't leak a file descriptor and a process in x11_maxsize_update()
if fgets() fails.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4147
Build 4190: arc lint + arc unit

Event Timeline

truckman updated this revision to Diff 17402.Jun 8 2016, 10:00 AM

truckman retitled this revision from to Fix file descriptor and memory leaks in libdpv.

truckman updated this object.

truckman edited the test plan for this revision. (Show Details)

truckman added a reviewer: dteske.

Herald added a subscriber: imp. · View Herald TranscriptJun 8 2016, 10:00 AM

Take

I've only a couple changes I'll make before committing with reference to OP. Instead of "doclose" I will use "opened" which makes the code easier to read.

That sounds reasonable to me.

Sorry for the delay, had a disk failure take out my operations for a full 4 weeks, but finally got all the data recovered and back up and running

Revision Contents
Changeset List

Path

Size

lib/

libdpv/

dialog_util.c

16 lines

Diff 17402

View Options

Fix file descriptor and memory leaks in libdpvNeeds ReviewPublicActions