diff options
| author | Richard Yao <[email protected]> | 2014-07-18 18:32:13 -0400 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2015-01-30 09:43:04 -0800 |
| commit | f9f431cd28ed6a78a2659758ce4a83c4c76fc3a1 (patch) | |
| tree | 72653a142ee6ccf7e19f8f74b9098eaf64792f29 /lib | |
| parent | 6466b61db6b4def6b077882ebbd4c3e83e917613 (diff) | |
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 <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #3054
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/libefi/rdwr_efi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/libefi/rdwr_efi.c b/lib/libefi/rdwr_efi.c index 147e30a37..24ec365b5 100644 --- 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; |