~ / search / GHSA-j3m6-gvm8-mhvw

GHSA-j3m6-gvm8-mhvw

No permission checks for editing/deleting records with CSV import form

Source: GitHub Advisory Database (reviewed) Moderate CVE-2023-49783

Packages

EcosystemPackageAffectedFixed in
Packagist silverstripe/admin >= 1.0.0, < 1.13.19 1.13.19
Packagist silverstripe/admin >= 2.0.0, < 2.1.8 2.1.8

Details

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

### Impact Users who don't have edit or delete permissions for records exposed in a `ModelAdmin` can still edit or delete records using the CSV import form, provided they have create permissions. The likelyhood of a user having create permissions but _not_ having edit or delete permissions is low, but it _is_ possible. Note that this doesn't affect any `ModelAdmin` which has had the import form disabled via the [`showImportForm` public property](https://api.silverstripe.org/4/SilverStripe/Admin/ModelAdmin.html#property_showImportForm), nor does it impact the `SecurityAdmin` section. #### Action may be required If you have a custom implementation of [`BulkLoader`](https://api.silverstripe.org/4/SilverStripe/Dev/BulkLoader.html), you should update your implementation to respect permissions when the return value of [`getCheckPermissions()`](https://api.silverstripe.org/4/SilverStripe/Dev/BulkLoader.html#method_getCheckPermissions) is true. If you are using any `BulkLoader` in your own project logic, or maintain a module which uses it, you should consider passing `true` to [`setCheckPermissions()`](https://api.silverstripe.org/4/SilverStripe/Dev/BulkLoader.html#method_setCheckPermissions) if the data is provided by users. **Base CVSS:** [4.3](https://nvd.nist.gov/vuln-metrics/cvss/v3-calculator?vector=AV:N/AC:L/PR:L/UI:N/S:U/C:N/I:L/A:N/E:F/RL:O/RC:C&version=3.1) **Reported by:** Guy Sartorelli from Silverstripe ### References - https://www.silverstripe.org/download/security-releases/CVE-2023-49783

References

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