Page MenuHomeFreeBSD

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

Authored by jaeyoon on Wed, Aug 5, 8:05 AM.

Details

Reviewers
imp
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 Skipped
Unit
Tests Skipped
Build Status
Buildable 75381
Build 72264: arc lint + arc unit