]> git.donarmstrong.com Git - roundcube.git/commitdiff
Imported Debian patch 0.2~alpha-4
authorVincent Bernat <bernat@debian.org>
Thu, 25 Dec 2008 10:38:13 +0000 (11:38 +0100)
committerJérémy Bobbio <lunar@debian.org>
Sat, 18 Jun 2011 17:58:45 +0000 (19:58 +0200)
15 files changed:
debian/changelog
debian/control
debian/control.in
debian/patches/correct-magic-path.patch
debian/patches/correct_install_path.patch
debian/patches/cve-2008-5620.patch [new file with mode: 0644]
debian/patches/dbconfig-common_support.patch
debian/patches/dont-use-preg-e-option.patch
debian/patches/fix_login.patch
debian/patches/series
debian/patches/use-db-backend.patch
debian/patches/use_packaged_tinymce.patch
debian/roundcube-core.cron.daily
debian/roundcube-core.postinst
debian/roundcube-core.postrm

index 4f0290e0b999f7f2846e60ba277edf6380ad0eeb..b97f2f8410c05af452acfa65868b7d944820683a 100644 (file)
@@ -1,3 +1,18 @@
+roundcube (0.2~alpha-4) experimental; urgency=low
+
+  * Add missing ${misc:Depends} to make Lintian happy.
+  * Add description to each patch.
+  * Execute cron job only if the directory to clean exists.
+  * Reload web server configuration instead of restart, thanks to a patch
+    from Tiago Bortoletto Vaz. Closes: #508633.
+  * Fix a vulnerability in quota image generation. This fixes
+    CVE-2008-5620. Thanks to Nico Golde for reporting it. Closes: #509596.
+  * Add missing dependency on php5-gd, used for quota bar.
+  * For roundcube-pgsql, depends on postgresql-client only. This package
+    is provided by the currently supported real package.
+
+ -- Vincent Bernat <bernat@debian.org>  Thu, 25 Dec 2008 11:38:13 +0100
+
 roundcube (0.2~alpha-3) experimental; urgency=high
 
   [ Vincent Bernat ]
index e1057b2b6e87e0aa1cf74747e19f57e3180185b4..a510c7941f65d826d1a549b7f51bc2eb87c0845e 100644 (file)
@@ -11,7 +11,7 @@ Vcs-Browser: http://svn.debian.org/wsvn/pkg-roundcube/roundcube
 
 Package: roundcube-core
 Architecture: all
-Depends: dbconfig-common, debconf | debconf-2.0, apache2 | lighttpd | httpd, php5, php5-mcrypt, roundcube-db, php-db, php-auth, php-net-smtp, php-net-socket, php-mail-mime (>= 1.5.0), ucf, tinymce (>= 3), ${misc:Depends}, libmagic1
+Depends: dbconfig-common, debconf | debconf-2.0, apache2 | lighttpd | httpd, php5, php5-mcrypt, php5-gd, roundcube-db, php-db, php-auth, php-net-smtp, php-net-socket, php-mail-mime (>= 1.5.0), ucf, tinymce (>= 3), ${misc:Depends}, libmagic1
 Replaces: roundcube
 Conflicts: roundcube (<< 0.1~rc2-2)
 Description: skinnable AJAX based webmail solution for IMAP servers
@@ -27,7 +27,7 @@ Description: skinnable AJAX based webmail solution for IMAP servers
 
 Package: roundcube
 Architecture: all
-Depends: roundcube-sqlite | roundcube-db, roundcube-core (= ${source:Version})
+Depends: roundcube-sqlite | roundcube-db, roundcube-core (= ${source:Version}), ${misc:Depends}
 Description: skinnable AJAX based webmail solution for IMAP servers
  RoundCube Webmail is a browser-based multilingual IMAP client with an
  application-like user interface. It provides full functionality
@@ -40,7 +40,7 @@ Description: skinnable AJAX based webmail solution for IMAP servers
 
 Package: roundcube-mysql
 Architecture: all
-Depends: php5-mysql, mysql-client | virtual-mysql-client
+Depends: php5-mysql, mysql-client | virtual-mysql-client, ${misc:Depends}
 Suggests: mysql-server
 Provides: roundcube-db
 Description: metapackage providing MySQL dependencies for RoundCube
@@ -50,7 +50,7 @@ Description: metapackage providing MySQL dependencies for RoundCube
 
 Package: roundcube-pgsql
 Architecture: all
-Depends: php5-pgsql, postgresql-client-8.1 | postgresql-client 
+Depends: php5-pgsql, postgresql-client, ${misc:Depends}
 Suggests: postgresql-server
 Provides: roundcube-db
 Description: metapackage providing PostgreSQL dependencies for RoundCube
@@ -60,7 +60,7 @@ Description: metapackage providing PostgreSQL dependencies for RoundCube
 
 Package: roundcube-sqlite
 Architecture: all
-Depends: php5-sqlite, sqlite
+Depends: php5-sqlite, sqlite, ${misc:Depends}
 Provides: roundcube-db
 Description: metapackage providing sqlite dependencies for RoundCube
  This package provides sqlite dependencies for RoundCube Webmail, a
index 76e727aa68fb9316feeaf2cf0a029d337cb1dab1..22c0cfc0d35bdcfb0ec4a76587b2e44ab31a6218 100644 (file)
@@ -11,7 +11,7 @@ Vcs-Browser: http://svn.debian.org/wsvn/pkg-roundcube/roundcube
 
 Package: roundcube-core
 Architecture: all
-Depends: dbconfig-common, debconf | debconf-2.0, apache2 | lighttpd | httpd, php5, php5-mcrypt, roundcube-db, php-db, php-auth, php-net-smtp, php-net-socket, php-mail-mime (>= 1.5.0), ucf, tinymce (>= 3), ${misc:Depends}, libmagic1
+Depends: dbconfig-common, debconf | debconf-2.0, apache2 | lighttpd | httpd, php5, php5-mcrypt, php5-gd, roundcube-db, php-db, php-auth, php-net-smtp, php-net-socket, php-mail-mime (>= 1.5.0), ucf, tinymce (>= 3), ${misc:Depends}, libmagic1
 Replaces: roundcube
 Conflicts: roundcube (<< 0.1~rc2-2)
 Description: skinnable AJAX based webmail solution for IMAP servers
@@ -27,7 +27,7 @@ Description: skinnable AJAX based webmail solution for IMAP servers
 
 Package: roundcube
 Architecture: all
-Depends: roundcube-sqlite | roundcube-db, roundcube-core (= ${source:Version})
+Depends: roundcube-sqlite | roundcube-db, roundcube-core (= ${source:Version}), ${misc:Depends}
 Description: skinnable AJAX based webmail solution for IMAP servers
  RoundCube Webmail is a browser-based multilingual IMAP client with an
  application-like user interface. It provides full functionality
@@ -40,7 +40,7 @@ Description: skinnable AJAX based webmail solution for IMAP servers
 
 Package: roundcube-mysql
 Architecture: all
-Depends: php5-mysql, mysql-client | virtual-mysql-client
+Depends: php5-mysql, mysql-client | virtual-mysql-client, ${misc:Depends}
 Suggests: mysql-server
 Provides: roundcube-db
 Description: metapackage providing MySQL dependencies for RoundCube
@@ -50,7 +50,7 @@ Description: metapackage providing MySQL dependencies for RoundCube
 
 Package: roundcube-pgsql
 Architecture: all
-Depends: php5-pgsql, postgresql-client-8.1 | postgresql-client 
+Depends: php5-pgsql, postgresql-client-8.1 | postgresql-client, ${misc:Depends}
 Suggests: postgresql-server
 Provides: roundcube-db
 Description: metapackage providing PostgreSQL dependencies for RoundCube
@@ -60,7 +60,7 @@ Description: metapackage providing PostgreSQL dependencies for RoundCube
 
 Package: roundcube-sqlite
 Architecture: all
-Depends: php5-sqlite, sqlite
+Depends: php5-sqlite, sqlite, ${misc:Depends}
 Provides: roundcube-db
 Description: metapackage providing sqlite dependencies for RoundCube
  This package provides sqlite dependencies for RoundCube Webmail, a
index e8e1b2e79d6679401af01e37a6189e558362228d..de9330f217e418d01c34fcaaa9ca7955d7ada2e9 100644 (file)
@@ -1,3 +1,5 @@
+On Debian, the magic file is in /usr/share/file/magic.
+
 --- roundcube-0.1/config/main.inc.php.dist~    2008-02-21 11:27:19.000000000 +0100
 +++ roundcube-0.1/config/main.inc.php.dist     2008-03-06 14:04:53.000000000 +0100
 @@ -277,7 +277,7 @@
index 1acfc5a578e14a5a51abdd7689dc1fde0094d37c..b8e6daa45abcec466b34ac85a7ecabf65bf0df84 100644 (file)
@@ -1,3 +1,5 @@
+Install path is /var/lib/roundcube for Debian. Don't try to guess it.
+
 --- a/program/include/iniset.php~      2008-06-09 22:57:53.000000000 +0200
 +++ a/program/include/iniset.php       2008-06-22 12:10:55.000000000 +0200
 @@ -27,7 +27,7 @@
diff --git a/debian/patches/cve-2008-5620.patch b/debian/patches/cve-2008-5620.patch
new file mode 100644 (file)
index 0000000..c1fdd23
--- /dev/null
@@ -0,0 +1,45 @@
+Fix CVE-2008-5620 which was caused by insufficient input sanitizing for quota bar.
+
+diff --git a/bin/quotaimg.php b/bin/quotaimg.php
+index 354f4eb..4e73c21 100644
+--- a/bin/quotaimg.php
++++ b/bin/quotaimg.php
+@@ -18,10 +18,10 @@
+ */
+-$used   = ((isset($_GET['u']) && !empty($_GET['u'])) || $_GET['u']=='0')?(int)$_GET['u']:'??';
+-$quota  = ((isset($_GET['q']) && !empty($_GET['q'])) || $_GET['q']=='0')?(int)$_GET['q']:'??';
+-$width  = empty($_GET['w']) ? 100 : (int)$_GET['w'];
+-$height = empty($_GET['h']) ? 14 : (int)$_GET['h'];
++$used   = isset($_GET['u']) ? intval($_GET['u']) : '??';
++$quota  = isset($_GET['q']) ? intval($_GET['q']) : '??';
++$width  = empty($_GET['w']) ? 100 : min(300, intval($_GET['w']));
++$height = empty($_GET['h']) ? 14  : min(50,  intval($_GET['h']));
+ /**
+  * Quota display
+@@ -159,7 +159,7 @@ function genQuota($used, $total, $width, $height)
+               }
+               $quota_width = $quota / 100 * $width;
+-              imagefilledrectangle($im, $border, 0, $quota, $height-2*$border, $fill);
++              imagefilledrectangle($im, $border, 0, $quota_width, $height-2*$border, $fill);
+               $string = $quota . '%';
+               $mid    = floor(($width-(strlen($string)*imagefontwidth($font)))/2)+1;
+@@ -178,6 +178,12 @@ function genQuota($used, $total, $width, $height)
+       imagedestroy($im);
+ }
+-genQuota($used, $quota, $width, $height);
++if ($width > 1 && $height > 1) {
++      genQuota($used, $quota, $width, $height);
++}
++else {
++      header("HTTP/1.0 404 Not Found");
++}
++
+ exit;
+ ?>
+\ No newline at end of file
index a4be31d02ac9648be7a44f42c530a7a97426e5a8..f6f01c28b59b547428bde3a4c1c3d6498b4abffa 100644 (file)
@@ -1,3 +1,5 @@
+The default db.inc.php is modified to adapt it to the use of dbconfig-common package.
+
 --- roundcube_0.1~beta2.2/config/db.inc.php.dist        2006-03-20 23:08:51.000000000 +0100
 +++ roundcube_0.1~beta2.2/config/db.inc.php.dist  2007-03-13 14:33:38.000000000 +0100
 @@ -14,13 +14,20 @@
index 1179a863955a2fad84a8aea4068a0268531be678..718526b26fca70839e08eedab15fe286428e46de 100644 (file)
@@ -1,3 +1,5 @@
+Fix a vulnerability due to the use of "e" option of preg_replace.
+
 --- roundcube-0.2~alpha/program/lib/html2text.php      2008-04-12 15:54:45.000000000 +0200
 +++ roundcube-0.2~alpha/program/lib/html2text.php      2008-12-13 14:21:44.000000000 +0100
 @@ -99,6 +99,22 @@
index b731043d40f96f244df4dac4599489f2fc8bb606..581deaf5846fb9cf7b9594bf20fe6be86f5472b6 100644 (file)
@@ -1,3 +1,5 @@
+Fix login redirection.
+
 --- a/program/include/rcmail.php~      2008-06-07 21:33:07.000000000 +0200
 +++ a/program/include/rcmail.php       2008-06-22 13:36:57.000000000 +0200
 @@ -474,7 +474,7 @@
index b68113a4f42d232eee453e70b77e4d6f0212ecad..07f681f3289391355aa4240e76138e51e5289a02 100644 (file)
@@ -5,3 +5,4 @@ use-db-backend.patch
 correct-magic-path.patch
 fix_login.patch
 dont-use-preg-e-option.patch
+cve-2008-5620.patch
index a32a0f45779238d6d4aa2aafb4d53d3ec1e387bd..d76df6d3d88743dd3ffbef671e452a6697fb409a 100644 (file)
@@ -1,3 +1,5 @@
+Use db backend since mdb2 is not yet available in Debian.
+
 --- roundcubemail-0.1-dep/config/db.inc.php.dist~      2008-03-03 22:32:15.000000000 +0100
 +++ roundcubemail-0.1-dep/config/db.inc.php.dist       2008-03-05 21:07:28.000000000 +0100
 @@ -27,7 +27,7 @@
index 72643fc96312250d70bf7f898a76de4646bc093f..290afb9fea1c93afc9a5b472436d38a2259e194a 100644 (file)
@@ -1,3 +1,5 @@
+Use tinymce from tinycme package instead of the shipped one.
+
 Index: roundcube-0.1~rc2/program/steps/mail/sendmail.inc
 ===================================================================
 --- a/program/steps/mail/sendmail.inc~ 2008-04-30 10:21:42.000000000 +0200
index 0fe51fc1e21364cf1e11a3a1f446d6912985adf5..ec4bfd77236e3d55d9c5e48c8283bb7c97757e46 100644 (file)
@@ -6,4 +6,6 @@ if [ -r /etc/default/roundcube-core ]; then
        . /etc/default/roundcube-core
 fi
 
-find /var/lib/roundcube/temp -type f -mtime +$MAX_TMPFILE_LIFETIME -print0 | xargs -0 -r rm
+if [ -d /var/lib/roundcube/temp ]; then
+       find /var/lib/roundcube/temp -type f -mtime +$MAX_TMPFILE_LIFETIME -print0 | xargs -0 -r rm
+fi
index 80a085aaaa2c5065473e49250dd3e95636ce4558..5e7e0e223a9f7a71b1c76dd7387eb67fe79a61cf 100644 (file)
@@ -151,10 +151,12 @@ EOF
        if [ "$res" = "true" ]; then
             for webserver in $restart; do
                webserver=${webserver%,}
+               # Redirection of 3 is needed because Debconf uses it and it might
+                # be inherited by webserver. See bug #446324.
                if [ -x /usr/sbin/invoke-rc.d ]; then
-                    invoke-rc.d $webserver restart
+                    invoke-rc.d $webserver reload 3>/dev/null || true
                else
-                    /etc/init.d/$webserver restart
+                    /etc/init.d/$webserver reload 3>/dev/null || true
                fi
             done
        fi
index 1e0467b93225dea7fa3a8e01facf61ff2a28a2c4..6659a77323de7f8f232ee8043f09ff70fbfdd33f 100644 (file)
@@ -73,10 +73,12 @@ case "$1" in
         if [ "$res" = "true" ]; then
             for webserver in $restart; do
                 webserver=${webserver%,}
+                # Redirection of 3 is needed because Debconf uses it and it might
+                # be inherited by webserver. See bug #446324.
                 if [ -x /usr/sbin/invoke-rc.d ]; then
-                    invoke-rc.d $webserver restart
+                    invoke-rc.d $webserver reload 3>/dev/null || true
                 else
-                    /etc/init.d/$webserver restart
+                    /etc/init.d/$webserver reload 3>/dev/null || true
                 fi
             done
         fi