mirror of
https://github.com/asterisk/asterisk.git
synced 2025-09-05 20:20:07 +00:00
Handle mpg123 failures without leaking fd's
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@1692 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
@@ -163,6 +163,8 @@ static int spawn_mp3(struct mohclass *class)
|
||||
#endif
|
||||
if (!files) {
|
||||
ast_log(LOG_WARNING, "Found no files in '%s'\n", class->dir);
|
||||
close(fds[0]);
|
||||
close(fds[1]);
|
||||
return -1;
|
||||
}
|
||||
class->pid = fork();
|
||||
@@ -189,6 +191,7 @@ static int spawn_mp3(struct mohclass *class)
|
||||
/* Check PATH as a last-ditch effort */
|
||||
execvp("mpg123", argv);
|
||||
ast_log(LOG_WARNING, "Exec failed: %s\n", strerror(errno));
|
||||
close(fds[1]);
|
||||
exit(1);
|
||||
} else {
|
||||
/* Parent */
|
||||
|
Reference in New Issue
Block a user