On Apr 4, 2012, at 3:00 PM, Mark Rustad wrote: > Add abort flag and use it to terminate processing when an exchange > is timed out or is reset. The abort flag is used in place of the > transport_generic_free_cmd function call in the reset and timeout > cases, because calling that function in that context would free > memory that was in use. The aborted flag allows the lifetime to > be managed in a more normal way, while truncating the processing. > > This change eliminates a source of memory corruption which > manifested in a variety of ugly ways. > > Signed-off-by: Mark Rustad <mark.d.rustad@xxxxxxxxx> > Acked-by: Kiran Patil <Kiran.patil@xxxxxxxxx> I should have said more about this originally. This makes it clearer what the consequences of not having the patch are. And of course your earlier comment to me that transport_generic_free_cmd can't be used there was exactly right. -- Mark Rustad, LAN Access Division, Intel Corporation -- 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
