From f143efa60c44c65c22aeeb04217f3501e3d04b22 Mon Sep 17 00:00:00 2001 From: Blue Swirl Date: Thu, 19 Aug 2010 20:24:43 +0000 Subject: Remove useless NULL check for qemu_strdup return value Found with this Coccinelle semantic patch: @@ expression E; identifier ptr; identifier fn ~= "qemu_strn?dup"; @@ -ptr = fn(E); -if (ptr == NULL) { ... } - +ptr = fn(E); Signed-off-by: Blue Swirl --- hw/virtio-9p-local.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/hw/virtio-9p-local.c b/hw/virtio-9p-local.c index 04f7f6f50..43c03c188 100644 --- a/hw/virtio-9p-local.c +++ b/hw/virtio-9p-local.c @@ -426,9 +426,6 @@ static int local_rename(FsContext *ctx, const char *oldpath, int err; tmp = qemu_strdup(rpath(ctx, oldpath)); - if (tmp == NULL) { - return -1; - } err = rename(tmp, rpath(ctx, newpath)); if (err == -1) { -- cgit v1.2.3