Page MenuHomeFreeBSD

ufshci: do not free the devq twice on SIM attach failure
ClosedPublic

Authored by jaeyoon on Wed, Aug 5, 8:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Aug 28, 1:13 PM
Unknown Object (File)
Fri, Aug 28, 6:37 AM
Unknown Object (File)
Thu, Aug 27, 10:11 PM
Unknown Object (File)
Wed, Aug 26, 7:07 PM
Unknown Object (File)
Wed, Aug 26, 5:00 PM
Unknown Object (File)
Wed, Aug 26, 5:00 PM
Unknown Object (File)
Tue, Aug 25, 12:54 PM
Unknown Object (File)
Mon, Aug 24, 11:04 PM
Subscribers
None

Details

Summary

cam_sim_free() with free_devq set already frees the devq, so the
following cam_simq_free() call on the xpt_bus_register() and
xpt_create_path() failure paths was a double free. Also clear
ctrlr->ufshci_sim so a later ufshci_sim_detach() does not operate on
the freed SIM.

Sponsored by: Samsung Electronics

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable