[BRLTTY] PATCH: Fix a file descriptor leak.

Joanmarie Diggs jdiggs at igalia.com
Fri Feb 22 14:53:45 EST 2013


On 02/22/2013 11:41 AM, Samuel Thibault wrote:
> Luke Yelavich, le Fri 22 Feb 2013 16:09:46 +0000, a écrit :
>>          brlapi_errfun="connect";
>>          setSocketErrno();
>> +        closeSocketDescriptor(handle->fileDescriptor);
>>          goto outlibc;
> 
> Mmm, the outlibc label is supposed to be doing it already, but not in
> this particular codepath indeed.
> 
> Could you try the attached patch instead?

That fixed it for me. Thanks!

Any chance we can get this into a stable release ASAP? Since GNOME's
universal access panel loads braille by default, more and more users are
going to run up against this.

Take care.
--joanie



More information about the BRLTTY mailing list