]> git.sesse.net Git - vlc/commitdiff
Always use the SIGPIPE hack even if not needed, it cannot hurt
authorRémi Denis-Courmont <remi@remlab.net>
Sun, 11 Apr 2010 09:07:46 +0000 (12:07 +0300)
committerRémi Denis-Courmont <remi@remlab.net>
Sun, 11 Apr 2010 09:07:46 +0000 (12:07 +0300)
bin/vlc.c

index 9ccbdd89401bcc2036c80f0b20ac506dbefa574b..5d6fec3502f6f8a9fccf05d7f10a647aded7ef81 100644 (file)
--- a/bin/vlc.c
+++ b/bin/vlc.c
@@ -54,11 +54,12 @@ extern char *FromLocale (const char *);
  *****************************************************************************/
 int main( int i_argc, const char *ppsz_argv[] )
 {
-#ifdef __APPLE__
-    /* The so-called POSIX-compliant MacOS X is not. 
-     * SIGPIPE fires even when it is blocked in all threads! */
+    /* The so-called POSIX-compliant MacOS X reportedly processes SIGPIPE even
+     * if it is blocked in all thread. Also some libraries want SIGPIPE blocked
+     * as they have no clue about signal masks.
+     * Note: this is NOT an excuse for not protecting against SIGPIPE. If
+     * LibVLC runs outside of VLC, we cannot rely on this code snippet. */
     signal (SIGPIPE, SIG_IGN);
-#endif
 
 #ifndef ALLOW_RUN_AS_ROOT
     if (geteuid () == 0)