os/xfree86: remove macro checking for POSIX symbols
We assume already that our X implementation is POSIX compliant anyway. So remove those redundant checking. SA_SIGINFO is left there. Signed-off-by: Tiago Vignatti <tiago.vignatti@nokia.com> Reviewed-by: Jeremy Huddleston <jeremyhu@apple.com> Reviewed-by: Jon TURNEY <jon.turney@dronecode.org.uk>
This commit is contained in:
parent
648c8871c9
commit
a9e6080dc9
|
@ -314,18 +314,10 @@ InstallSignalHandlers(void)
|
||||||
signal(SIGEMT, SIG_DFL);
|
signal(SIGEMT, SIG_DFL);
|
||||||
#endif
|
#endif
|
||||||
signal(SIGFPE, SIG_DFL);
|
signal(SIGFPE, SIG_DFL);
|
||||||
#ifdef SIGBUS
|
|
||||||
signal(SIGBUS, SIG_DFL);
|
signal(SIGBUS, SIG_DFL);
|
||||||
#endif
|
|
||||||
#ifdef SIGSYS
|
|
||||||
signal(SIGSYS, SIG_DFL);
|
signal(SIGSYS, SIG_DFL);
|
||||||
#endif
|
|
||||||
#ifdef SIGXCPU
|
|
||||||
signal(SIGXCPU, SIG_DFL);
|
signal(SIGXCPU, SIG_DFL);
|
||||||
#endif
|
|
||||||
#ifdef SIGXFSZ
|
|
||||||
signal(SIGXFSZ, SIG_DFL);
|
signal(SIGXFSZ, SIG_DFL);
|
||||||
#endif
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -863,9 +855,7 @@ OsVendorInit(void)
|
||||||
{
|
{
|
||||||
static Bool beenHere = FALSE;
|
static Bool beenHere = FALSE;
|
||||||
|
|
||||||
#ifdef SIGCHLD
|
|
||||||
signal(SIGCHLD, SIG_DFL); /* Need to wait for child processes */
|
signal(SIGCHLD, SIG_DFL); /* Need to wait for child processes */
|
||||||
#endif
|
|
||||||
|
|
||||||
if (!beenHere) {
|
if (!beenHere) {
|
||||||
umask(022);
|
umask(022);
|
||||||
|
|
|
@ -157,9 +157,7 @@ xf86writeConfigFile (const char *filename, XF86ConfigPtr cptr)
|
||||||
|
|
||||||
#if !defined(HAS_SAVED_IDS_AND_SETEUID)
|
#if !defined(HAS_SAVED_IDS_AND_SETEUID)
|
||||||
/* Need to fork to change ruid without loosing euid */
|
/* Need to fork to change ruid without loosing euid */
|
||||||
#ifdef SIGCHLD
|
|
||||||
csig = signal(SIGCHLD, SIG_DFL);
|
csig = signal(SIGCHLD, SIG_DFL);
|
||||||
#endif
|
|
||||||
switch ((pid = fork()))
|
switch ((pid = fork()))
|
||||||
{
|
{
|
||||||
case -1:
|
case -1:
|
||||||
|
@ -180,9 +178,7 @@ xf86writeConfigFile (const char *filename, XF86ConfigPtr cptr)
|
||||||
p = waitpid(pid, &status, 0);
|
p = waitpid(pid, &status, 0);
|
||||||
} while (p == -1 && errno == EINTR);
|
} while (p == -1 && errno == EINTR);
|
||||||
}
|
}
|
||||||
#ifdef SIGCHLD
|
|
||||||
signal(SIGCHLD, csig);
|
signal(SIGCHLD, csig);
|
||||||
#endif
|
|
||||||
if (p != -1 && WIFEXITED(status) && WEXITSTATUS(status) == 0)
|
if (p != -1 && WIFEXITED(status) && WEXITSTATUS(status) == 0)
|
||||||
return 1; /* success */
|
return 1; /* success */
|
||||||
else
|
else
|
||||||
|
|
|
@ -168,15 +168,9 @@ OsInit(void)
|
||||||
struct sigaction act, oact;
|
struct sigaction act, oact;
|
||||||
int i;
|
int i;
|
||||||
int siglist[] = { SIGSEGV, SIGQUIT, SIGILL, SIGFPE, SIGBUS,
|
int siglist[] = { SIGSEGV, SIGQUIT, SIGILL, SIGFPE, SIGBUS,
|
||||||
#ifdef SIGSYS
|
|
||||||
SIGSYS,
|
SIGSYS,
|
||||||
#endif
|
|
||||||
#ifdef SIGXCPU
|
|
||||||
SIGXCPU,
|
SIGXCPU,
|
||||||
#endif
|
|
||||||
#ifdef SIGXFSZ
|
|
||||||
SIGXFSZ,
|
SIGXFSZ,
|
||||||
#endif
|
|
||||||
#ifdef SIGEMT
|
#ifdef SIGEMT
|
||||||
SIGEMT,
|
SIGEMT,
|
||||||
#endif
|
#endif
|
||||||
|
|
21
os/utils.c
21
os/utils.c
|
@ -1122,10 +1122,7 @@ XNFstrdup(const char *s)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
#ifdef SIGVTALRM
|
|
||||||
#define SMART_SCHEDULE_POSSIBLE
|
#define SMART_SCHEDULE_POSSIBLE
|
||||||
#endif
|
|
||||||
|
|
||||||
#ifdef SMART_SCHEDULE_POSSIBLE
|
#ifdef SMART_SCHEDULE_POSSIBLE
|
||||||
#define SMART_SCHEDULE_SIGNAL SIGALRM
|
#define SMART_SCHEDULE_SIGNAL SIGALRM
|
||||||
#define SMART_SCHEDULE_TIMER ITIMER_REAL
|
#define SMART_SCHEDULE_TIMER ITIMER_REAL
|
||||||
|
@ -1211,30 +1208,18 @@ OsBlockSignals (void)
|
||||||
sigset_t set;
|
sigset_t set;
|
||||||
|
|
||||||
sigemptyset (&set);
|
sigemptyset (&set);
|
||||||
#ifdef SIGALRM
|
|
||||||
sigaddset (&set, SIGALRM);
|
sigaddset (&set, SIGALRM);
|
||||||
#endif
|
|
||||||
#ifdef SIGVTALRM
|
|
||||||
sigaddset (&set, SIGVTALRM);
|
sigaddset (&set, SIGVTALRM);
|
||||||
#endif
|
|
||||||
#ifdef SIGWINCH
|
#ifdef SIGWINCH
|
||||||
sigaddset (&set, SIGWINCH);
|
sigaddset (&set, SIGWINCH);
|
||||||
#endif
|
#endif
|
||||||
#ifdef SIGIO
|
#ifdef SIGIO
|
||||||
sigaddset (&set, SIGIO);
|
sigaddset (&set, SIGIO);
|
||||||
#endif
|
#endif
|
||||||
#ifdef SIGTSTP
|
|
||||||
sigaddset (&set, SIGTSTP);
|
sigaddset (&set, SIGTSTP);
|
||||||
#endif
|
|
||||||
#ifdef SIGTTIN
|
|
||||||
sigaddset (&set, SIGTTIN);
|
sigaddset (&set, SIGTTIN);
|
||||||
#endif
|
|
||||||
#ifdef SIGTTOU
|
|
||||||
sigaddset (&set, SIGTTOU);
|
sigaddset (&set, SIGTTOU);
|
||||||
#endif
|
|
||||||
#ifdef SIGCHLD
|
|
||||||
sigaddset (&set, SIGCHLD);
|
sigaddset (&set, SIGCHLD);
|
||||||
#endif
|
|
||||||
sigprocmask (SIG_BLOCK, &set, &PreviousSignalMask);
|
sigprocmask (SIG_BLOCK, &set, &PreviousSignalMask);
|
||||||
}
|
}
|
||||||
#endif
|
#endif
|
||||||
|
@ -1280,21 +1265,17 @@ int
|
||||||
System(char *command)
|
System(char *command)
|
||||||
{
|
{
|
||||||
int pid, p;
|
int pid, p;
|
||||||
#ifdef SIGCHLD
|
|
||||||
void (*csig)(int);
|
void (*csig)(int);
|
||||||
#endif
|
|
||||||
int status;
|
int status;
|
||||||
|
|
||||||
if (!command)
|
if (!command)
|
||||||
return 1;
|
return 1;
|
||||||
|
|
||||||
#ifdef SIGCHLD
|
|
||||||
csig = signal(SIGCHLD, SIG_DFL);
|
csig = signal(SIGCHLD, SIG_DFL);
|
||||||
if (csig == SIG_ERR) {
|
if (csig == SIG_ERR) {
|
||||||
perror("signal");
|
perror("signal");
|
||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
#endif
|
|
||||||
|
|
||||||
#ifdef DEBUG
|
#ifdef DEBUG
|
||||||
ErrorF("System: `%s'\n", command);
|
ErrorF("System: `%s'\n", command);
|
||||||
|
@ -1317,12 +1298,10 @@ System(char *command)
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
#ifdef SIGCHLD
|
|
||||||
if (signal(SIGCHLD, csig) == SIG_ERR) {
|
if (signal(SIGCHLD, csig) == SIG_ERR) {
|
||||||
perror("signal");
|
perror("signal");
|
||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
#endif
|
|
||||||
|
|
||||||
return p == -1 ? -1 : status;
|
return p == -1 ? -1 : status;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue