Merge lp:~clint-fewbar/ubuntu/lucid/php5/lucid-sru-lp564920 into lp:ubuntu/lucid-updates/php5

Proposed by Clint Byrum on 2010-08-16
Status: Merged
Merge reported by: Thierry Carrez
Merged at revision: not available
Proposed branch: lp:~clint-fewbar/ubuntu/lucid/php5/lucid-sru-lp564920
Merge into: lp:ubuntu/lucid-updates/php5
Diff against target: 51 lines (+31/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/lp564920-fix-big-files.patch (+23/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~clint-fewbar/ubuntu/lucid/php5/lucid-sru-lp564920
Reviewer Review Type Date Requested Status
Thierry Carrez Approve on 2010-09-06
Chuck Short 2010-08-16 Pending
Ubuntu branches 2010-08-16 Pending
Review via email: mp+32803@code.launchpad.net

Description of the Change

SRU for LP #564920

To post a comment you must log in.
Thierry Carrez (ttx) wrote :

Looks good, however before this can be committed there should be an SRU report on bug 564920. This allows the SRU team to assess impact vs. regression risk, and give a testcase for the lucid-proposed testers. See https://wiki.ubuntu.com/StableReleaseUpdates#Procedure for details...

Thierry Carrez (ttx) wrote :

SRU report is ready, uploading in a few.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-05-10 04:00:03 +0000
3+++ debian/changelog 2010-08-16 19:33:47 +0000
4@@ -1,3 +1,10 @@
5+php5 (5.3.2-1ubuntu4.3) lucid-proposed; urgency=low
6+
7+ * debian/patches/lp564920-fix-big-files.patch: Fix downloading of large
8+ files (LP: #564920)
9+
10+ -- Clint Byrum <clint@ubuntu.com> Mon, 16 Aug 2010 11:50:19 -0700
11+
12 php5 (5.3.2-1ubuntu4.2) lucid-proposed; urgency=low
13
14 * debian/patches/session_save_path.patch: Save PHP sessions to
15
16=== added file 'debian/patches/lp564920-fix-big-files.patch'
17--- debian/patches/lp564920-fix-big-files.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/lp564920-fix-big-files.patch 2010-08-16 19:33:47 +0000
19@@ -0,0 +1,23 @@
20+Description: don't mmap large files
21+Author: Marc Deslauriers <marc.deslauriers@ubuntu.com>
22+Bug: http://bugs.php.net/bug.php?id=52102
23+Ubuntu-Bug: https://bugs.edge.launchpad.net/ubuntu/+source/php5/+bug/564920
24+
25+diff -Nur php5-5.3.2/main/streams/plain_wrapper.c php5-5.3.2.new/main/streams/plain_wrapper.c
26+--- php5-5.3.2/main/streams/plain_wrapper.c 2010-02-23 10:04:29.000000000 -0500
27++++ php5-5.3.2.new/main/streams/plain_wrapper.c 2010-07-14 18:24:13.000000000 -0400
28+@@ -627,7 +627,13 @@
29+
30+ switch (value) {
31+ case PHP_STREAM_MMAP_SUPPORTED:
32+- return fd == -1 ? PHP_STREAM_OPTION_RETURN_ERR : PHP_STREAM_OPTION_RETURN_OK;
33++ if (fd == -1)
34++ return PHP_STREAM_OPTION_RETURN_ERR;
35++ /* Don't mmap large files */
36++ do_fstat(data, 1);
37++ if (data->sb.st_size > 4 * 1024 * 1024)
38++ return PHP_STREAM_OPTION_RETURN_ERR;
39++ return PHP_STREAM_OPTION_RETURN_OK;
40+
41+ case PHP_STREAM_MMAP_MAP_RANGE:
42+ do_fstat(data, 1);
43
44=== modified file 'debian/patches/series'
45--- debian/patches/series 2010-05-10 04:00:03 +0000
46+++ debian/patches/series 2010-08-16 19:33:47 +0000
47@@ -54,3 +54,4 @@
48 enchant_unaligned_memory_access.patch
49 fix-mysql-badmem.patch
50 session_save_path.patch
51+lp564920-fix-big-files.patch

Subscribers

People subscribed via source and target branches

to all changes: