[zbin] Fix size used for memset in alloc_output_file

The output->buf field is a pointer, not an array, so sizeof() is not
applicable.  We must use the allocated string length instead.

Identified by gcc:

  util/zbin.c: In function ‘alloc_output_file’:
  util/zbin.c:146:37: warning: argument to ‘sizeof’ in ‘memset’ call
    is the same expression as the destination; did you mean to
    dereference it? [-Wsizeof-pointer-memaccess]
  memset ( output->buf, 0xff, sizeof ( output->buf ) );

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
Signed-off-by: Michael Brown <mcb30@ipxe.org>

diff --git a/src/util/zbin.c b/src/util/zbin.c
index 0dabaf1e3ad1b47387dc474d0d4d850261d458c0..3b7cf95b3df7491dc3c5ef78f087bdfd1f3de2a6 100644 (file)
--- a/src/util/zbin.c
+++ b/src/util/zbin.c
@@ -143,7 +143,7 @@ static int alloc_output_file ( size_t max_len, struct output_file *output ) {
                          max_len );
                return -1;
        }
-       memset ( output->buf, 0xff, sizeof ( output->buf ) );
+       memset ( output->buf, 0xff, max_len );
        return 0;
 }