Merge lp:~james-page/ubuntu/maverick/tomcat6/fix-654549 into lp:ubuntu/maverick-updates/tomcat6

Proposed by James Page
Status: Merged
Merged at revision: 27
Proposed branch: lp:~james-page/ubuntu/maverick/tomcat6/fix-654549
Merge into: lp:ubuntu/maverick-updates/tomcat6
Diff against target: 32 lines (+11/-3)
2 files modified
debian/changelog (+8/-0)
debian/tomcat6.postinst (+3/-3)
To merge this branch: bzr merge lp:~james-page/ubuntu/maverick/tomcat6/fix-654549
Reviewer Review Type Date Requested Status
Marc Deslauriers Approve
Review via email: mp+58080@code.launchpad.net

Description of the change

Minor change to maintainer script to resolve upgrade failures on Maverick where '/' is used in JAVA_OPTS

To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Patch looks good. ACK. Uploading now for SRU.

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 2011-03-24 10:10:09 +0000
3+++ debian/changelog 2011-04-18 07:32:39 +0000
4@@ -1,3 +1,11 @@
5+tomcat6 (6.0.28-2ubuntu1.3) maverick-proposed; urgency=low
6+
7+ * Fix update failures when JAVA_OPTS contains / (LP: #654549)
8+ - debian/tomcat6.postinst: amended sed calls to use % instead of / when
9+ generating /etc/default/tomcat6.
10+
11+ -- James Page <james.page@canonical.com> Fri, 15 Apr 2011 12:30:47 +0100
12+
13 tomcat6 (6.0.28-2ubuntu1.2) maverick-security; urgency=low
14
15 * SECURITY UPDATE: directory traversal via incorrect ServetContext
16
17=== modified file 'debian/tomcat6.postinst'
18--- debian/tomcat6.postinst 2010-08-25 09:07:03 +0000
19+++ debian/tomcat6.postinst 2011-04-18 07:32:39 +0000
20@@ -18,9 +18,9 @@
21 tmpfile=`mktemp /tmp/tomcat6.XXXXXXXXXX`
22 chmod 644 $tmpfile
23 cat $TEMPLATE \
24- | sed "s/^TOMCAT6_USER=.*$/TOMCAT6_USER=$TOMCAT6_USER/" \
25- | sed "s/^TOMCAT6_GROUP=.*$/TOMCAT6_GROUP=$TOMCAT6_GROUP/" \
26- | sed "s/^JAVA_OPTS=.*$/JAVA_OPTS=\"$JAVA_OPTS\"/" \
27+ | sed "s%^TOMCAT6_USER=.*$%TOMCAT6_USER=$TOMCAT6_USER%" \
28+ | sed "s%^TOMCAT6_GROUP=.*$%TOMCAT6_GROUP=$TOMCAT6_GROUP%" \
29+ | sed "s%^JAVA_OPTS=.*$%JAVA_OPTS=\"$JAVA_OPTS\"%" \
30 >> $tmpfile
31 ucf --debconf-ok --sum-file /usr/share/tomcat6/defaults.md5sum $tmpfile $CONFFILE
32 rm -f $tmpfile

Subscribers

People subscribed via source and target branches

to all changes: