[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v1 1/1] hexdump: Add null guard on output file
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-trivial] [PATCH v1 1/1] hexdump: Add null guard on output file. |
Date: |
Sat, 02 Aug 2014 17:26:13 +0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.6.0 |
31.07.2014 04:31, Peter Crosthwaite wrote:
> To avoid callsites with optional output having to NULL guard.
Isn't it a bit backwards? If we don't need output, maybe we should
not call hexdump in the first place?
Thanks,
/mjt
> Signed-off-by: Peter Crosthwaite <address@hidden>
> ---
> Noting in-tree is affected by this yet, but I though I'd get this
> out of the way straight-up rather than elongate other series.
>
> util/hexdump.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/util/hexdump.c b/util/hexdump.c
> index 969b340..b607236 100644
> --- a/util/hexdump.c
> +++ b/util/hexdump.c
> @@ -19,6 +19,9 @@ void qemu_hexdump(const char *buf, FILE *fp, const char
> *prefix, size_t size)
> {
> unsigned int b;
>
> + if (!fp) {
> + return;
> + }
> for (b = 0; b < size; b++) {
> if ((b % 16) == 0) {
> fprintf(fp, "%s: %04x:", prefix, b);
>
- Re: [Qemu-trivial] [PATCH v1 1/1] hexdump: Add null guard on output file.,
Michael Tokarev <=