devs/foxbox
1
0
Fork 0
forked from len0rd/rockbox
Code Pull requests Activity

apps: fix close of negative fd in talk.c

Browse source 
Bug reported by UBSan.

Change-Id: I02276521510acac8696499895fec15993c55aaf6
This commit is contained in:
Aidan MacDonald 2022-05-02 16:34:18 +01:00
parent e5e457b526
commit dadfbfc6a5
1 changed files with 5 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

9
apps/talk.c
View file

@ -552,8 +552,8 @@ alloc_err:
}
static inline int load_voicefile_failure(int fd)
{
/*if (fd >= 0) probably redundant */
close(fd);
if (fd >= 0)
close(fd);
return -1;
}
/* load the voice file into the mp3 buffer */
@ -829,7 +829,7 @@ void talk_init(void)
* and so we can re-use it if it's already allocated in any event */
filehandle = open_voicefile();
if (filehandle > -1)
if (filehandle >= 0)
{
if (!load_voicefile_index(filehandle))
{
@ -880,7 +880,8 @@ void talk_init(void)
voice_thread_init();
out:
close(filehandle); /* close again, this was just to detect presence */
if (filehandle >= 0)
close(filehandle); /* close again, this was just to detect presence */
}
/* somebody else claims the mp3 buffer, e.g. for regular play/record */