On Tue, Jan 20, 2015 at 12:31:32PM -0500, Jeff Cody wrote: > The backing_filename string in mirror_run() is only used to check > for a NULL string, so we don't need to allocate 1024 bytes (or, later, > PATH_MAX bytes), when we only need to copy the first 2 characters. > > We technically only need 1 byte, as we are just checking for NULL, but > since backing_filename[] is populated by bdrv_get_backing_filename(), a > string size of 1 will always only return '\0'; > > Signed-off-by: Jeff Cody <jc...@redhat.com> > --- > block/mirror.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
pgpcDXzK7TJwe.pgp
Description: PGP signature
