Use (void) memcpy(), not (void *) memcpy()

This was caught by Clang.  Clearly the intent of this code was
to explicitly ignore the return value.

Signed-off-by: Richard Yao <ryao@gentoo.org>
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #3054

diff --git a/lib/libefi/rdwr_efi.c b/lib/libefi/rdwr_efi.c
index 147e30a374adc8e2c55e3d64c1c76927274d0f32..24ec365b5a9fd1c71331345252d4b5a24b14874e 100644 (file)
--- a/lib/libefi/rdwr_efi.c
+++ b/lib/libefi/rdwr_efi.c
@@ -861,11 +861,11 @@ write_pmbr(int fd, struct dk_gpt *vtoc)
        /* LINTED -- always longlong aligned */
        dk_ioc.dki_data = (efi_gpt_t *)buf;
        if (efi_ioctl(fd, DKIOCGETEFI, &dk_ioc) == -1) {
-               (void *) memcpy(&mb, buf, sizeof (mb));
+               (void) memcpy(&mb, buf, sizeof (mb));
                bzero(&mb, sizeof (mb));
                mb.signature = LE_16(MBB_MAGIC);
        } else {
-               (void *) memcpy(&mb, buf, sizeof (mb));
+               (void) memcpy(&mb, buf, sizeof (mb));
                if (mb.signature != LE_16(MBB_MAGIC)) {
                        bzero(&mb, sizeof (mb));
                        mb.signature = LE_16(MBB_MAGIC);
@@ -905,7 +905,7 @@ write_pmbr(int fd, struct dk_gpt *vtoc)
                *cp++ = 0xff;
        }
 
-       (void *) memcpy(buf, &mb, sizeof (mb));
+       (void) memcpy(buf, &mb, sizeof (mb));
        /* LINTED -- always longlong aligned */
        dk_ioc.dki_data = (efi_gpt_t *)buf;
        dk_ioc.dki_lba = 0;