[dm-devel] dm-mpath: Fix setup_scsi_dh()
Bart Van Assche
bvanassche at acm.org
Mon Sep 17 15:34:15 UTC 2018
On 9/17/18 8:11 AM, Mike Snitzer wrote:
> Moving kfree() into setup_scsi_dh() would require use of a common goto
> for cleanup (something parse_path() already has with 'goto bad;').
>
> But having kfree() in parse_path() is cleaner/symmetric since call to
> scsi_dh_attached_handler_name() -- and associated memory allocation --
> occurs in parse_path().
>
> If you're OK with this, I'll get a proper patch staged based on your
> header and obviously add a Reported-by attributed to you.
That sounds fine to me.
Thanks!
Bart.
More information about the dm-devel
mailing list