On 02/08/2017 11:24 PM, Bart Van Assche wrote: > Use the se_cmd reference counting mechanism instead of calling the > .release_cmd() method directly. Fix a reference leak triggered by > transport_cmd_check_stop_to_fabric() that did not matter until > now because of the direct .release_cmd() calls. If CMD_T_STOP is > set for a command, transport_cmd_finish_abort() namely calls > transport_cmd_check_stop_to_fabric() and the latter function > returns 1 although the command refcount is not yet zero. > > Signed-off-by: Bart Van Assche <bart.vanassche@xxxxxxxxxxx> > Cc: Hannes Reinecke <hare@xxxxxxxx> > Cc: Christoph Hellwig <hch@xxxxxx> > Cc: Andy Grover <agrover@xxxxxxxxxx> > Cc: David Disseldorp <ddiss@xxxxxxx> > --- > drivers/target/target_core_transport.c | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > Reviewed-by: Hannes Reinecke <hare@xxxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxxx +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
