[koha-commits] main Koha release repository branch, 3.2.x, updated. v3.02.03-82-gbc60c23

Git repo owner gitmaster at git.koha-community.org
Thu Feb 24 20:27:06 CET 2011


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 3.2.x has been updated
       via  bc60c233601e34a75545ac5767a6a486d8c2c348 (commit)
       via  348546aef192beb8aea09dd7bda60debfe7e1b5f (commit)
       via  49f8964bb29a380fae393d2946dedb567e482802 (commit)
       via  44038d38fd1e72d0fa82e4a135ea58b2028832fa (commit)
       via  799509d90b08a95d0f24293b006f05a658cb4ade (commit)
      from  5294c412c8af4cf52878071076a0b9a54e118bea (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bc60c233601e34a75545ac5767a6a486d8c2c348
Author: Chris Nighswonger <cnighswonger at foundations.edu>
Date:   Thu Feb 24 09:57:11 2011 -0500

    Security Bugfix: Bug 1953 Adding Placeholders to SQL To Avoid Potential Injection Attacks
    
    This patch addresses both security issues mentioned in the summary of the report
    submitted by Frère Sébastien Marie included below.
    
    ---------------------------
    The problem is here: 'C4/AuthoritiesMarc.pm' in the function 'DelAuthority':
    The argument $authid is included directly (not via statement) in the SQL.
    
    For the exploit of this problem, you can use 'authorities/authorities-home.pl'
    with authid on the URL and op=delete (something like
    "authorities/authorities-home.pl?op=delete&authid=xxx").
    
    This should successfully call DelAuthority, without authentification...
    (DelAuthority is call BEFORE get_template_and_user, so before authentification
    [This should be an issue also...]).
    
    Please note that the problem isn't only that anyone can delete an authority of
    this choose, it is more general: with "authid=1%20or%1=1" (after inclusion sql
    will be like: "delete from auth_header where authid=1 or 1=1") you delete all
    authorities ; with "authid=1;delete%20from%xxx" it is "delete from auth_header
    where authid=1;delete from xxx" and so delete what you want...
    
    SQL-INJECTION is very permissive: you can redirect the output in a file (with
    some MySQL function), so write thea file of you choose in the server, in order
    to create a backdoor, and compromise the server.
    
    Signed-off-by: Frère Sébastien Marie <semarie-koha at latrappe.fr>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    (cherry picked from commit b0f60221f41041665c4fecacce35654fc8d45a01)
    
    Signed-off-by: Chris Nighswonger <chris.nighswonger at gmail.com>

commit 348546aef192beb8aea09dd7bda60debfe7e1b5f
Author: marcel at libdevelop.rijksmuseum.nl <marcel at libdevelop.rijksmuseum.nl>
Date:   Mon Feb 21 15:40:44 2011 +0100

    2742: Wrong language name in the preferences
    
    Follow up patch. Improvement suggested by Belgian translators (Hans Supply).
    
    Signed-off-by: Frédéric Demians <f.demians at tamil.fr>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    (cherry picked from commit fc1b7201c55d00d6c43a0ecc04bee09cd0e0bde7)
    
    Signed-off-by: Chris Nighswonger <chris.nighswonger at gmail.com>

commit 49f8964bb29a380fae393d2946dedb567e482802
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Fri Feb 11 22:22:16 2011 -0500

    Partial fix for Bug 5745, Overdues with fines report not showing titles
    
    - Adding title, subtitle, and author to output
    - Reworking display of shelving location selection
    
    Patch does not address the contents of 'overdue status' and 'notified by'
    
    Signed-off-by: Nicole Engard <nengard at bywatersolutions.com>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    (cherry picked from commit 50c0ae09a12e10b851022d6b3fe18aa0b345a98b)
    
    Signed-off-by: Chris Nighswonger <chris.nighswonger at gmail.com>

commit 44038d38fd1e72d0fa82e4a135ea58b2028832fa
Author: Paul Poulain <paul.poulain at biblibre.com>
Date:   Tue Feb 15 13:52:35 2011 +1300

    Bug 5759 : displaying 2nd email if there is one on print template for borrowers
    
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    (cherry picked from commit 2f9bebf802530ad1a96fe5bb5ced1b8ef02ba46f)
    
    Signed-off-by: Chris Nighswonger <chris.nighswonger at gmail.com>

commit 799509d90b08a95d0f24293b006f05a658cb4ade
Author: Owen Leonard <oleonard at myacpl.org>
Date:   Thu Feb 17 13:32:32 2011 -0500

    Fix for Bug 5769 - notice tab disappearing on edit patron
    
    Signed-off-by: Nicole Engard <nengard at bywatersolutions.com>
    Signed-off-by: Chris Cormack <chrisc at catalyst.net.nz>
    (cherry picked from commit ed2b583eb10cb81c26c43714572bbb2c85d5797c)
    
    Signed-off-by: Chris Nighswonger <chris.nighswonger at gmail.com>

-----------------------------------------------------------------------

Summary of changes:
 C4/AuthoritiesMarc.pm                              |    4 +-
 C4/Overdues.pm                                     |    2 +
 authorities/authorities-home.pl                    |    5 +---
 circ/branchoverdues.pl                             |    9 ++++++-
 .../prog/en/includes/members-menu.inc              |   10 +++---
 koha-tmpl/intranet-tmpl/prog/en/modules/about.tmpl |    4 +-
 .../prog/en/modules/circ/branchoverdues.tmpl       |   27 +++++--------------
 .../prog/en/modules/members/moremember-print.tmpl  |    3 +-
 8 files changed, 29 insertions(+), 35 deletions(-)


hooks/post-receive
-- 
main Koha release repository


More information about the koha-commits mailing list