Your message dated Sat, 19 Jul 2014 12:29:19 +0100 with message-id <1405769359.9607.4.camel@jacala.jungle.funky-badger.org> and subject line Re: Bug#742637: squeeze-pu: package spip/2.1.1-3squeeze9 has caused the Debian Bug report #742637, regarding squeeze-pu: package spip/2.1.1-3squeeze9 to be marked as done. This means that you claim that the problem has been dealt with. If this is not the case it is now your responsibility to reopen the Bug report if necessary, and/or fix the problem forthwith. (NB: If you are a system administrator and have no idea what this message is talking about, this may indicate a serious mail system misconfiguration somewhere. Please contact owner@bugs.debian.org immediately.) -- 742637: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=742637 Debian Bug Tracking System Contact owner@bugs.debian.org with problems
--- Begin Message ---
- To: Debian Bug Tracking System <submit@bugs.debian.org>
- Subject: squeeze-pu: package spip/2.1.1-3squeeze9
- From: David Prévot <taffit@debian.org>
- Date: Tue, 25 Mar 2014 15:54:12 -0400
- Message-id: <20140325195412.GA22148@persil.tilapin.org>
Package: release.debian.org Severity: normal Tags: squeeze User: release.debian.org@packages.debian.org Usertags: pu Dear release team, As agreed with the security team, here I am proposing another oldstable update for fixing minor security issues in SPIP. Attached debdiff, thanks in advance for considering. Regards David -- System Information: Debian Release: jessie/sid APT prefers unstable APT policy: (500, 'unstable'), (500, 'testing'), (110, 'experimental') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 3.13-trunk-amd64 (SMP w/2 CPU cores) Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dashdiff --git a/debian/changelog b/debian/changelog index 0427965..25a8296 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,11 @@ +spip (2.1.1-3squeeze9) squeeze; urgency=medium + + * Fix missing escape + * Fix missing escape in SQL + * Update security screen to 1.1.9 (Closes: #742456) + + -- David Prévot <taffit@debian.org> Thu, 13 Mar 2014 16:58:14 -0300 + spip (2.1.1-3squeeze8) squeeze; urgency=medium * Document fixed #729172 diff --git a/debian/patches/fix_missing_escape.patch b/debian/patches/fix_missing_escape.patch new file mode 100644 index 0000000..7d08c36 --- /dev/null +++ b/debian/patches/fix_missing_escape.patch @@ -0,0 +1,19 @@ +From: Cédric Morin <cedric.morin@yterium.com> +Subject: Fix missing escape + + * ecrire/inc/log.php: Always escape < in logs, unless the _LOG_BRUT + constant is defined to true at the end of debug. + +Origin: upstream, http://core.spip.org/projects/spip/repository/revisions/21251 +Bug-Debian: http://bugs.debian.org/742456 +--- a/ecrire/inc/log.php ++++ b/ecrire/inc/log.php +@@ -56,7 +56,7 @@ + + $f = @fopen($logfile, "ab"); + if ($f) { +- fputs($f, ($logname!==NULL) ? $m : str_replace('<','<',$m)); ++ fputs($f, (defined('_LOG_BRUT') AND _LOG_BRUT) ? $m : str_replace('<','<',$m)); + fclose($f); + } + diff --git a/debian/patches/fix_missing_escape_in_SQL.patch b/debian/patches/fix_missing_escape_in_SQL.patch new file mode 100644 index 0000000..8024698 --- /dev/null +++ b/debian/patches/fix_missing_escape_in_SQL.patch @@ -0,0 +1,18 @@ +From: Cédric Morin <cedric.morin@yterium.com> +Subject: Fix missing escape in SQL + + * ecrire/inc/quete.php: Fix missing escape in a SQL request. + +Origin: upstream, http://core.spip.org/projects/spip/repository/revisions/21247 +Bug-Debian: http://bugs.debian.org/742456 +--- a/ecrire/public/quete.php ++++ b/ecrire/public/quete.php +@@ -248,7 +248,7 @@ + $exposer[$m][$type][$principal] = true; + if ($type == 'id_mot'){ + if (!$parent) { +- $parent = sql_getfetsel('id_groupe','spip_mots',"id_mot=" . $principal, '','','','',$connect); ++ $parent = sql_getfetsel('id_groupe','spip_mots',"id_mot=" . intval($principal), '','','','',$connect); + } + if ($parent) + $exposer[$m]['id_groupe'][$parent] = true; diff --git a/debian/patches/series b/debian/patches/series index 7646257..1ec8a6c 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -19,3 +19,5 @@ fix_privilege_escalation_2.1.21.patch fix_CSRF_on_logout.patch fix_XSS_on_author_page.patch fix_XSS_on_signature.patch +fix_missing_escape.patch +fix_missing_escape_in_SQL.patch diff --git a/debian/security/ecran_securite.php b/debian/security/ecran_securite.php index 7e6ec1c..36b0044 100644 --- a/debian/security/ecran_securite.php +++ b/debian/security/ecran_securite.php @@ -5,7 +5,7 @@ * ------------------ */ -define('_ECRAN_SECURITE', '1.1.8'); // 2013-08-29 +define('_ECRAN_SECURITE', '1.1.9'); // 2014-03-13 /* * Documentation : http://www.spip.net/fr_article4200.html @@ -254,13 +254,12 @@ if (isset($_REQUEST['connect']) AND // cas qui permettent de sortir d'un commentaire PHP (strpos($_REQUEST['connect'], "?")!==false + OR strpos($_REQUEST['connect'], "<")!==false OR strpos($_REQUEST['connect'], ">")!==false OR strpos($_REQUEST['connect'], "\n")!==false OR strpos($_REQUEST['connect'], "\r")!==false) ) { - $_REQUEST['connect'] = str_replace(array("?", ">", "\r", "\n"), "", $_REQUEST['connect']); - if (isset($_GET['connect'])) $_GET['connect'] = $_REQUEST['connect']; - if (isset($_POST['connect'])) $_POST['connect'] = $_REQUEST['connect']; + $ecran_securite_raison = "malformed connect argument"; } /* @@ -317,4 +316,4 @@ if ( } -?> +?> \ No newline at end of fileAttachment: signature.asc
Description: Digital signature
--- End Message ---
--- Begin Message ---
- To: 742637-done@bugs.debian.org
- Cc: David Prévot <taffit@debian.org>
- Subject: Re: Bug#742637: squeeze-pu: package spip/2.1.1-3squeeze9
- From: "Adam D. Barratt" <adam@adam-barratt.org.uk>
- Date: Sat, 19 Jul 2014 12:29:19 +0100
- Message-id: <1405769359.9607.4.camel@jacala.jungle.funky-badger.org>
- In-reply-to: <1395869662.18435.12.camel@jacala.jungle.funky-badger.org>
- References: <20140325195412.GA22148@persil.tilapin.org> <1395778023.625.10.camel@jacala.jungle.funky-badger.org> <8590257bb8da6a38ca5c9a8eb8698a26.squirrel@webmail.tilapin.org> <1395869662.18435.12.camel@jacala.jungle.funky-badger.org>
Control: tags -1 -pending +wontfix On Wed, 2014-03-26 at 21:34 +0000, Adam D. Barratt wrote: > Control: tags -1 + pending > > On Tue, 2014-03-25 at 18:02 -0400, David Prévot wrote: > > > Please go ahead > > > > Thanks Adam for your quick ACK, uploaded. > > Flagged for acceptance; thanks. This update was technically included in 6.0.10, but the package was then removed at the security team's request (see #753502). Regards, Adam
--- End Message ---