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

Proposed by Chen-Han Hsiao (Stanley)
Status: Merged
Approved by: Michael Terry
Approved revision: 34
Merge reported by: Michael Terry
Merged at revision: not available
Proposed branch: lp:~swem/ubuntu/wily/lshw/lp1471983
Merge into: lp:ubuntu/wily-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/wily/lshw/lp1471983
Reviewer Review Type Date Requested Status
Michael Terry Approve
Review via email: mp+274074@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
Michael Terry (mterry) wrote :

Looks good to me. Uploaded to wily.

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 2015-08-18 15:06:49 +0000
3+++ debian/changelog 2015-10-11 13:48:56 +0000
4@@ -1,3 +1,10 @@
5+lshw (02.17-1.1ubuntu3) wily; 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 21:35:54 +0800
11+
12 lshw (02.17-1.1ubuntu2) wily; urgency=medium
13
14 [ Manoj Iyer ]
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:48:56 +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-18 15:06:49 +0000
53+++ debian/patches/series 2015-10-11 13:48:56 +0000
54@@ -16,3 +16,4 @@
55 Do-not-brutally-merge-NICs-in-the-same-PCI-function-.patch
56 smbios-noscan.patch
57 fat-inspection.patch
58+presumably-fix-653.patch

Subscribers

People subscribed via source and target branches