[FFmpeg-devel] [PATCH 1/2] ffmpeg: reset tty state correctly

Nicolas George george at nsup.org
Mon Jul 27 10:44:54 CEST 2015


L'octidi 8 thermidor, an CCXXIII, Ganesh Ajjanagadde a écrit :
> +            abort();

Sorry to not have noticed it before:

abort() is not in the list of async-signal-safe functions, see:
http://pubs.opengroup.org/onlinepubs/009695399/functions/xsh_chap02_04.html

But the specification for abort() tells us: "The SIGABRT signal shall be
sent to the calling process as if by means of raise() with the argument
SIGABRT.", and raise() is in the list.

The authors must have been sleepy and forgotten abort(), it is probably ok.

(But it should not be necessary anyway.)

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150727/517ee47d/attachment.sig>


More information about the ffmpeg-devel mailing list