[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