[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 06/10] vl: exit if we are not able to write into
From: |
Kevin Wolf |
Subject: |
[Qemu-devel] Re: [PATCH 06/10] vl: exit if we are not able to write into the pipe |
Date: |
Wed, 03 Mar 2010 14:51:17 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.7) Gecko/20100120 Fedora/3.0.1-1.fc12 Thunderbird/3.0.1 |
Am 03.03.2010 13:06, schrieb Juan Quintela:
> Signed-off-by: Juan Quintela <address@hidden>
> ---
> vl.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index db7a178..119c7e4 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -5745,6 +5745,7 @@ int main(int argc, char **argv, char **envp)
> uint8_t status = 1;
> if (write(fds[1], &status, 1) != 1) {
> perror("daemonize. Writing to pipe\n");
> + exit(1);
> }
> } else
> #endif
This is already the handling code for a fatal error, it's just trying to
tell the parent process that it went wrong. Completing the context:
fprintf(stderr, "Could not acquire pid file: %s\n",
strerror(errno));
exit(1);
}
So there is already an exit(1), this patch doesn't change anything.
Kevin
- [Qemu-devel] [PATCH 00/10] FORTIFY_SOURCE followup, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 01/10] cow: return errno instead of -1, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 02/10] slirp: check system() success, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 03/10] qcow2: return errno instead of -1, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 04/10] qcow: return errno instead of -1, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 05/10] vmdk: return errno instead of -1, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 06/10] vl: exit if we are not able to write into the pipe, Juan Quintela, 2010/03/03
- [Qemu-devel] Re: [PATCH 06/10] vl: exit if we are not able to write into the pipe,
Kevin Wolf <=
- [Qemu-devel] [PATCH 07/10] vmdk: make vmdk_snapshot_create return -errno, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 08/10] vmdk: fix double free, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 09/10] vmdk: share cleanup code, Juan Quintela, 2010/03/03
- [Qemu-devel] [PATCH 10/10] block: print errno on error, Juan Quintela, 2010/03/03