Impact
The BulkAssetsController::update() method accepts company_id directly from user input without calling Company::getIdForCurrentUser(), the standard company-scoping function used by every other controller in the codebase. A non-superadmin user can move assets across company boundaries, breaking multi-tenancy isolation.
Patches
Patched in grokability/snipe-it@d58fda6
References
Impact
The
BulkAssetsController::update()method acceptscompany_iddirectly from user input without callingCompany::getIdForCurrentUser(), the standard company-scoping function used by every other controller in the codebase. A non-superadmin user can move assets across company boundaries, breaking multi-tenancy isolation.Patches
Patched in grokability/snipe-it@d58fda6
References