Summary: | Missing placeholders in admin/authorised_values.pl | ||
---|---|---|---|
Product: | Koha | Reporter: | Adrien SAURAT <adrien.saurat> |
Component: | System Administration | Assignee: | Adrien SAURAT <adrien.saurat> |
Status: | CLOSED FIXED | QA Contact: | Paul Poulain <paul.poulain> |
Severity: | critical | ||
Priority: | P5 - low | CC: | gmcharlt, katrin.fischer, paul.poulain |
Version: | 3.6 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Bug Depends on: | |||
Bug Blocks: | 7013 | ||
Attachments: |
proposed patch
[SIGNED-OFF] Bug 7450: missing placeholders in authorised_values.pl [SIGNED-OFF] Bug 7450: missing placeholders in authorised_values.pl |
Description
Adrien SAURAT
2012-01-17 10:29:35 UTC
Created attachment 7197 [details] [review] proposed patch Created attachment 7280 [details] [review] [SIGNED-OFF] Bug 7450: missing placeholders in authorised_values.pl Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Successfully added, edited and deleted authorised values. No problems found. Created attachment 7281 [details] [review] [SIGNED-OFF] Bug 7450: missing placeholders in authorised_values.pl Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Successfully added, edited and deleted authorised values. No problems found. QA comment * in fact this patch solves a security issue, of a missing SQL placeholder. Upgrading severity * tiny patch, passed QA patch pushed, with a follow-up to remove an unconditionnal warn that was already here (detected by my pre-applypatch git hook) This fix was included in the 3.6.x branch prior to 3.6.4. |