security-and-privacy/gnupg
security-and-privacy/gnupg
1
0
Fork 0
mirror of git://git.gnupg.org/gnupg.git synced 2024-06-15 00:29:49 +02:00
Code Activity

Fix killing PID -1.

Browse Source 
When the KILLSCD command had been sent a race condition would occur
causing PID -1 getting killed, which on Linux seems to terminate all
applications for the current user.
This commit is contained in:
Ben Kibbey 2012-04-09 20:52:22 -04:00 committed by Werner Koch
parent f1e1387bee
commit bee0ac28c9
1 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
scd/apdu.c
View File

@ -1016,7 +1016,8 @@ pcsc_get_status_wrapped (int slot, unsigned int *status)
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
return sw; return sw;
@ -1180,7 +1181,8 @@ pcsc_send_apdu_wrapped (int slot, unsigned char *apdu, size_t apdulen,
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
return sw; return sw;
@ -1321,7 +1323,8 @@ control_pcsc_wrapped (int slot, unsigned long ioctl_code,
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
return pcsc_error_to_sw (err); return pcsc_error_to_sw (err);
@ -1422,7 +1425,8 @@ close_pcsc_reader_wrapped (int slot)
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
return 0; return 0;
@ -1640,7 +1644,8 @@ reset_pcsc_reader_wrapped (int slot)
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
return sw; return sw;
@ -1965,7 +1970,8 @@ open_pcsc_reader_wrapped (const char *portstr)
close (slotp->pcsc.rsp_fd); close (slotp->pcsc.rsp_fd);
slotp->pcsc.req_fd = -1; slotp->pcsc.req_fd = -1;
slotp->pcsc.rsp_fd = -1; slotp->pcsc.rsp_fd = -1;
kill (slotp->pcsc.pid, SIGTERM); if (slotp->pcsc.pid != -1)
kill (slotp->pcsc.pid, SIGTERM);
slotp->pcsc.pid = (pid_t)(-1); slotp->pcsc.pid = (pid_t)(-1);
slotp->used = 0; slotp->used = 0;
/* There is no way to return SW. */ /* There is no way to return SW. */