~ / search / GHSA-5hfv-c864-qcq9

GHSA-5hfv-c864-qcq9

CI4MS has a Deactivated User Session Bypass (active=0)

Source: GitHub Advisory Database (reviewed) Moderate CVE-2026-41891

Packages

EcosystemPackageAffectedFixed in
Packagist ci4-cms-erp/ci4ms >= 0.26.0, < 0.31.8.0 0.31.8.0

Details

As written in the advisory (Markdown source, shown as plain text).

### Summary The auth filter has the deactivated/banned user check commented out. ### Details CodeIgniter Shield's `loggedIn()` re-checks the `status` field (catching `status='banned'`), but does **not** re-check the `active` field for existing sessions. When an admin deactivates a user (`active=0`) after they have already logged in: - Their session cookie remains valid - `auth()->loggedIn()` still returns `true` - The commented-out code is the only mechanism that would have checked `!$user->active` ### Evidence <img width="981" height="654" alt="image" src="https://github.com/user-attachments/assets/6f75d144-5bcf-4a3f-bc35-bb0715c3ed05" /> ### Impact - User deactivation does NOT immediately revoke backend access - Deactivated user retains full access until session expires (default: 7200s) ### Additional note The commented-out block appears to be a deferred placeholder — it was written but disabled from the very first commit that introduced the filter, and has never been active. The later addition of SessionTracker (v0.31.4.0) suggests the dev was aware of the session revocation gap, but account-level deactivation (users.active = 0) remains unenforced. Could you verify if this is intentionally pending or simply forgotten and not documented?.

References

External links from the advisory. They lead to third-party sites.