Opened 8 years ago
Closed 8 years ago
#6193 closed defect (fixed)
apps/gdalserver.c uses signal definitions without signal.h
Reported by: | gdt | Owned by: | warmerdam |
---|---|---|---|
Priority: | normal | Milestone: | 2.0.2 |
Component: | ConfigBuild | Version: | unspecified |
Severity: | normal | Keywords: | |
Cc: |
Description
According to POSIX, one must include signal.h when using signal(3).
Attachments (1)
Change History (5)
by , 8 years ago
Attachment: | signal-patch.txt added |
---|
comment:1 by , 8 years ago
comment:2 by , 8 years ago
Propose change to cpl_spawn.cpp rather than port.
-
cpl_spawn.cpp
34 34 #include "cpl_string.h" 35 35 #include "cpl_multiproc.h" 36 36 37 #ifdef SIGPIPE 38 # include <csignal> 39 #endif 40 37 41 #define PIPE_BUFFER_SIZE 4096 38 42 39 43 #define IN_FOR_PARENT 0
comment:3 by , 8 years ago
Sorry, I see you meant gdalserver.c. Ignore my last. I've applied the patch to gdalserver.c but doing it by hand.
The line numbers in the patch are bad. If not all systems define SIGPIPE, we may need a #ifdefined around
if( !bFork ) signal(SIGPIPE, SIG_IGN);
comment:4 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
+1 on the bug, but I am working on an alternate patch