sg: add missing free(msg); in ->errdetails() handler

msg gets malloc'd unconditionally, so we need to free msg if we're
not using msg for ret (strdup isn't really needed here to begin with).

Signed-off-by: Tomohiro Kusumi <tkusumi@tuxera.com>
Signed-off-by: Jens Axboe <axboe@fb.com>

diff --git a/engines/sg.c b/engines/sg.c
index 2148e87c190f8c1dfc4f648aecb99796888ca777..9d339bd345554e671395e9c96fc13c66c0c14dab 100644 (file)
--- a/engines/sg.c
+++ b/engines/sg.c
@@ -759,8 +759,10 @@ static char *fio_sgio_errdetails(struct io_u *io_u)
                }
        }
 
                }
        }
 

-       if (!ret)
+       if (!ret) {

                ret = strdup("SG Driver did not report a Host, Driver or Device check");
                ret = strdup("SG Driver did not report a Host, Driver or Device check");

+               free(msg);
+       }

 
        return ret;
 }
 
        return ret;
 }