Merge ~oddbloke/ubuntu/+source/tmux:lp1835439 into ubuntu/+source/tmux:ubuntu/bionic-devel

Proposed by Dan Watkins
Status: Merged
Approved by: Andreas Hasenack
Approved revision: e81a8076730e30085e459930396cdfe98f3796d5
Merged at revision: e81a8076730e30085e459930396cdfe98f3796d5
Proposed branch: ~oddbloke/ubuntu/+source/tmux:lp1835439
Merge into: ubuntu/+source/tmux:ubuntu/bionic-devel
Diff against target: 76 lines (+43/-1)
4 files modified
debian/changelog (+8/-0)
debian/control (+2/-1)
debian/patches/series (+1/-0)
debian/patches/tmux-format_expand-leak.patch (+32/-0)
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
Canonical Server Pending
git-ubuntu developers Pending
Review via email: mp+369731@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

- d/changelog correct for SRU
- bug has SRU template filled out

+1

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Just noticed that update-maintainer wasn't run in the previous upload (to update d/control), could you please run it and include that change in this mp?

Revision history for this message
Andreas Hasenack (ahasenack) :
review: Needs Fixing
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Sponsoring e81a8076730e30085e459930396cdfe98f3796d5

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Tagged and uploaded:

$ git push pkg upload/2.6-3ubuntu0.2
Enumerating objects: 25, done.
Counting objects: 100% (25/25), done.
Delta compression using up to 2 threads
Compressing objects: 100% (19/19), done.
Writing objects: 100% (19/19), 5.18 KiB | 1.29 MiB/s, done.
Total 19 (delta 11), reused 0 (delta 0)
To ssh://git.launchpad.net/~usd-import-team/ubuntu/+source/tmux
 * [new tag] upload/2.6-3ubuntu0.2 -> upload/2.6-3ubuntu0.2

$ dput ubuntu ../tmux_2.6-3ubuntu0.2_source.changes
Checking signature on .changes
gpg: ../tmux_2.6-3ubuntu0.2_source.changes: Valid signature from AC983EB5BF6BCBA9
Checking signature on .dsc
gpg: ../tmux_2.6-3ubuntu0.2.dsc: Valid signature from AC983EB5BF6BCBA9
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading tmux_2.6-3ubuntu0.2.dsc: done.
  Uploading tmux_2.6-3ubuntu0.2.debian.tar.xz: done.
  Uploading tmux_2.6-3ubuntu0.2_source.buildinfo: done.
  Uploading tmux_2.6-3ubuntu0.2_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 10e86fb..4602a9e 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,11 @@
6+tmux (2.6-3ubuntu0.2) bionic; urgency=medium
7+
8+ * d/p/tmux-format_expand-leak.patch: Do not leak memory when working out job
9+ name in formats. (LP: #1835439)
10+ * Update maintainer to reflect Ubuntu delta.
11+
12+ -- Daniel Watkins <oddbloke@ubuntu.com> Thu, 04 Jul 2019 15:45:07 -0400
13+
14 tmux (2.6-3ubuntu0.1) bionic; urgency=medium
15
16 * d/p/tmux-pane-border-status-leak.patch: Fixed memory leak in
17diff --git a/debian/control b/debian/control
18index 69370cf..6e0e4d7 100644
19--- a/debian/control
20+++ b/debian/control
21@@ -1,7 +1,8 @@
22 Source: tmux
23 Section: admin
24 Priority: optional
25-Maintainer: Romain Francoise <rfrancoise@debian.org>
26+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
27+XSBC-Original-Maintainer: Romain Francoise <rfrancoise@debian.org>
28 Build-Depends: debhelper (>= 8.9.4~),
29 dh-autoreconf,
30 dpkg-dev (>= 1.16.1~),
31diff --git a/debian/patches/series b/debian/patches/series
32index ca10374..87d4f66 100644
33--- a/debian/patches/series
34+++ b/debian/patches/series
35@@ -1,2 +1,3 @@
36 platform-quirks.diff
37 tmux-pane-border-status-leak.patch
38+tmux-format_expand-leak.patch
39diff --git a/debian/patches/tmux-format_expand-leak.patch b/debian/patches/tmux-format_expand-leak.patch
40new file mode 100644
41index 0000000..e27b74a
42--- /dev/null
43+++ b/debian/patches/tmux-format_expand-leak.patch
44@@ -0,0 +1,32 @@
45+Description: Do not leak memory when working out job name in formats.
46+Origin: upstream, https://github.com/tmux/tmux/commit/ab6f0bb3485f6626bad6f585e4d17059fe4f7152
47+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/tmux/+bug/1835439
48+Last-Update: 2019-07-04
49+---
50+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
51+
52+--- a/format.c
53++++ b/format.c
54+@@ -1102,7 +1102,7 @@
55+ char *
56+ format_expand(struct format_tree *ft, const char *fmt)
57+ {
58+- char *buf, *out;
59++ char *buf, *out, *name;
60+ const char *ptr, *s, *saved = fmt;
61+ size_t off, len, n, outlen;
62+ int ch, brackets;
63+@@ -1141,8 +1141,11 @@
64+
65+ if (ft->flags & FORMAT_NOJOBS)
66+ out = xstrdup("");
67+- else
68+- out = format_job_get(ft, xstrndup(fmt, n));
69++ else {
70++ name = xstrndup(fmt, n);
71++ out = format_job_get(ft, name);
72++ free(name);
73++ }
74+ outlen = strlen(out);
75+
76+ while (len - off < outlen + 1) {

Subscribers

People subscribed via source and target branches