shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-01-26 14:04:15 +03:00
Code Releases Activity

Alasdair correctly pointed out that if the two closes are concurrent (I haven't

Browse Source 
checked, so they *might*), there is still a race possibility with the last
fix. This patch fixes that.
This commit is contained in:
Petr Rockai 2010-10-27 11:40:14 +00:00
parent d9f958852c
commit ab98a1c5dc
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
daemons/clvmd/clvmd.c
View File

@ -188,6 +188,14 @@ static void child_init_signal(int status)
exit(status);
}
static void safe_close(int *fd)
{
if (*fd >= 0) {
int to_close = *fd;
*fd = -1;
close(to_close);
}
}
void debuglog(const char *fmt, ...)
{
@ -838,10 +846,7 @@ static void main_loop(int local_sock, int cmd_timeout)
lastfd->next = thisfd->next;
free_fd = thisfd;
thisfd = lastfd;
if (free_fd->fd >= 0) {
close(free_fd->fd);
free_fd->fd = -1;
}
safe_close(&(free_fd->fd));
/* Queue cleanup, this also frees the client struct */
add_to_lvmqueue(free_fd, NULL, 0, NULL);
@ -1091,10 +1096,7 @@ static int read_from_local_sock(struct local_client *thisfd)
thisfd->bits.localsock.pipe_client->bits.pipe.client =
NULL;
if (thisfd->fd >= 0) {
close(thisfd->fd);
thisfd->fd = -1;
}
safe_close(&(thisfd->fd));
return 0;
} else {
int comms_pipe[2];