Merge lp:~swem/ubuntu/vivid/lshw/lp1471983 into lp:ubuntu/vivid-proposed/lshw

Proposed by Chen-Han Hsiao (Stanley)
Status: Rejected
Rejected by: Sebastien Bacher
Proposed branch: lp:~swem/ubuntu/vivid/lshw/lp1471983
Merge into: lp:ubuntu/vivid-proposed/lshw
Diff against target: 58 lines (+38/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/presumably-fix-653.patch (+30/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~swem/ubuntu/vivid/lshw/lp1471983
Reviewer Review Type Date Requested Status
Sebastien Bacher Disapprove
Review via email: mp+274073@code.launchpad.net

Description of the change

For bug 1471983, I also encounter this issue in Ubuntu OEM project.

This bug is fixed in https://github.com/lyonel/lshw/commit/d048d300b5daeb44887a7fc06ddeb120119cac8a

The issue in upstream: [lshw segfaults, with some 16GB USB-3 sticks from Patriot] http://www.ezix.org/project/ticket/653

I've made a patch to fix it.

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

the fix has been uploaded to wily, vivid is not the current stable anymore so not likely important for a SRU, let's skip this one. Please let me know if you really need it in vivid though

review: Disapprove

Unmerged revisions

34. By Chen-Han Hsiao (Stanley)

[d048d30] Fix lshw crash issue on system with some USB-3 sticks
(LP: #1471983)

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 2015-08-24 16:11:32 +0000
3+++ debian/changelog 2015-10-11 13:45:05 +0000
4@@ -1,3 +1,10 @@
5+lshw (02.17-1.1ubuntu1.2) vivid; urgency=medium
6+
7+ * [d048d30] Fix lshw crash issue on system with some USB-3 sticks
8+ (LP: #1471983)
9+
10+ -- Chen-Han Hsiao (Stanley) <stanley.hsiao@canonical.com> Sun, 11 Oct 2015 20:54:12 +0800
11+
12 lshw (02.17-1.1ubuntu1.1) vivid; urgency=medium
13
14 * Backport cpuinfo support for aarch64 (LP: #1485086)
15
16=== added file 'debian/patches/presumably-fix-653.patch'
17--- debian/patches/presumably-fix-653.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/presumably-fix-653.patch 2015-10-11 13:45:05 +0000
19@@ -0,0 +1,30 @@
20+From d048d300b5daeb44887a7fc06ddeb120119cac8a Mon Sep 17 00:00:00 2001
21+From: Lyonel Vincent <lyonel@ezix.org>
22+Date: Sun, 13 Jul 2014 11:21:59 +0000
23+Subject: [PATCH] (presumably) fix #653
24+
25+git-svn-id: http://ezix.org/source/packages/lshw/development@2535 811e2811-9fd9-0310-a116-b6e8ac943c8b
26+---
27+ src/core/scsi.cc | 6 +++---
28+ 1 file changed, 3 insertions(+), 3 deletions(-)
29+
30+diff --git a/src/core/scsi.cc b/src/core/scsi.cc
31+index ed059cb..d85fcc8 100644
32+--- a/src/core/scsi.cc
33++++ b/src/core/scsi.cc
34+@@ -481,9 +481,9 @@ hwNode & node)
35+ memset(rsp_buff, 0, sizeof(rsp_buff));
36+ if (do_inq(sg_fd, 0, 1, 0x80, rsp_buff, MX_ALLOC_LEN, 0))
37+ {
38+- len = rsp_buff[3];
39+- if (len > 0)
40+- node.setSerial(hw::strip(string(rsp_buff + 4, len)));
41++ char _len = rsp_buff[3];
42++ if (_len > 0)
43++ node.setSerial(hw::strip(string(rsp_buff + 4, _len)));
44+ }
45+
46+ memset(rsp_buff, 0, sizeof(rsp_buff));
47+--
48+1.9.1
49+
50
51=== modified file 'debian/patches/series'
52--- debian/patches/series 2015-08-24 16:11:32 +0000
53+++ debian/patches/series 2015-10-11 13:45:05 +0000
54@@ -16,3 +16,4 @@
55 smbios-noscan.patch
56 fat-inspection.patch
57 cpuinfo-arm-aarch64-support.patch
58+presumably-fix-653.patch

Subscribers

People subscribed via source and target branches