Skip to content

Commit 1b58d24

Browse files
martinpittmartinpitt
committed
umockdev-run: Don't strip off trailing whitespace from sysfs attributes
libusb 1.0.24 changed the sysfs parser to require attributes be terminated with a newline [1]. Update umockdev-run to keep trailing whitespace (in particular, newlines) unmodified, so that the mocked testbed reproduces them faithfully. Do the minimal changes to the pre-recorded devices to get along with the new libusb. Adjust /umockdev-testbed-usb/lsusb accordingly to terminate busnum/devnum with line feeds. Fixes #115 [1] libusb/libusb@c3864c822b
1 parent b35ae31 commit 1b58d24

File tree

5 files changed

+32
-32
lines changed

5 files changed

+32
-32
lines changed

‎devices/cameras/canon-powershot-sx200.umockdev‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,11 @@ A: bNumConfigurations=1
3838
A: bNumInterfaces= 1
3939
A: bcdDevice=0002
4040
A: bmAttributes=c0
41-
A: busnum=1
41+
A: busnum=1\n
4242
A: configuration=
4343
H: descriptors=1201000200000040A904C03102000102030109022700010100C001090400000306010100070581020002000705020200020007058303080009
4444
A: dev=189:10
45-
A: devnum=11
45+
A: devnum=11\n
4646
A: devpath=1.5.2.3
4747
A: idProduct=31c0
4848
A: idVendor=04a9
@@ -90,11 +90,11 @@ A: bNumConfigurations=1
9090
A: bNumInterfaces= 1
9191
A: bcdDevice=0100
9292
A: bmAttributes=e0
93-
A: busnum=1
93+
A: busnum=1\n
9494
A: configuration=
9595
H: descriptors=12010002090001400904580000010102000109021900010100E0320904000001090000000705810301000C
9696
A: dev=189:4
97-
A: devnum=5
97+
A: devnum=5\n
9898
A: devpath=1.5.2
9999
A: idProduct=0058
100100
A: idVendor=0409
@@ -141,11 +141,11 @@ A: bNumConfigurations=1
141141
A: bNumInterfaces= 1
142142
A: bcdDevice=0001
143143
A: bmAttributes=e0
144-
A: busnum=1
144+
A: busnum=1\n
145145
A: configuration=
146146
H: descriptors=1201000209000240EF17051001000000000109022900010100E0010904000001090001000705810301000C0904000101090002000705810301000C
147147
A: dev=189:2
148-
A: devnum=3
148+
A: devnum=3\n
149149
A: devpath=1.5
150150
A: idProduct=1005
151151
A: idVendor=17ef
@@ -190,11 +190,11 @@ A: bNumConfigurations=1
190190
A: bNumInterfaces= 1
191191
A: bcdDevice=0000
192192
A: bmAttributes=e0
193-
A: busnum=1
193+
A: busnum=1\n
194194
A: configuration=
195195
H: descriptors=12010002090001408780200000000000000109021900010100E0000904000001090000000705810301000C
196196
A: dev=189:1
197-
A: devnum=2
197+
A: devnum=2\n
198198
A: devpath=1
199199
A: idProduct=0020
200200
A: idVendor=8087
@@ -241,11 +241,11 @@ A: bNumConfigurations=1
241241
A: bNumInterfaces= 1
242242
A: bcdDevice=0305
243243
A: bmAttributes=e0
244-
A: busnum=1
244+
A: busnum=1\n
245245
A: configuration=
246246
H: descriptors=12010002090000406B1D020005030302010109021900010100E0000904000001090000000705810304000C
247247
A: dev=189:0
248-
A: devnum=1
248+
A: devnum=1\n
249249
A: devpath=0
250250
A: idProduct=0002
251251
A: idVendor=1d6b

‎devices/input/usbkbd.umockdev‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,11 @@ A: bNumConfigurations=1
125125
A: bNumInterfaces= 2
126126
A: bcdDevice=0320
127127
A: bmAttributes=a0
128-
A: busnum=1
128+
A: busnum=1\n
129129
A: configuration=
130130
H: descriptors=1201100100000008F305070020030000000109023B00020100A020090400000103010100092100012101223F000705810308000809040100010300000009210001000122640007058203040008
131131
A: dev=189:8
132-
A: devnum=9
132+
A: devnum=9\n
133133
A: devpath=1.5.4.2
134134
L: driver=../../../../../../../../bus/usb/drivers/usb
135135
A: idProduct=0007
@@ -183,11 +183,11 @@ A: bNumConfigurations=1
183183
A: bNumInterfaces= 1
184184
A: bcdDevice=0320
185185
A: bmAttributes=a0
186-
A: busnum=1
186+
A: busnum=1\n
187187
A: configuration=
188188
H: descriptors=1201100109000008F305810020030102000109021900010100A019090400000109000000070581030100FF
189189
A: dev=189:6
190-
A: devnum=7
190+
A: devnum=7\n
191191
A: devpath=1.5.4
192192
L: driver=../../../../../../../bus/usb/drivers/usb
193193
A: idProduct=0081
@@ -242,11 +242,11 @@ A: bNumConfigurations=1
242242
A: bNumInterfaces= 1
243243
A: bcdDevice=0001
244244
A: bmAttributes=e0
245-
A: busnum=1
245+
A: busnum=1\n
246246
A: configuration=
247247
H: descriptors=1201000209000240EF17051001000000000109022900010100E0010904000001090001000705810301000C0904000101090002000705810301000C
248248
A: dev=189:3
249-
A: devnum=4
249+
A: devnum=4\n
250250
A: devpath=1.5
251251
L: driver=../../../../../../bus/usb/drivers/usb
252252
A: idProduct=1005
@@ -300,11 +300,11 @@ A: bNumConfigurations=1
300300
A: bNumInterfaces= 1
301301
A: bcdDevice=0000
302302
A: bmAttributes=e0
303-
A: busnum=1
303+
A: busnum=1\n
304304
A: configuration=
305305
H: descriptors=12010002090001408780200000000000000109021900010100E0000904000001090000000705810301000C
306306
A: dev=189:1
307-
A: devnum=2
307+
A: devnum=2\n
308308
A: devpath=1
309309
L: driver=../../../../../bus/usb/drivers/usb
310310
A: idProduct=0020
@@ -360,11 +360,11 @@ A: bNumConfigurations=1
360360
A: bNumInterfaces= 1
361361
A: bcdDevice=0310
362362
A: bmAttributes=e0
363-
A: busnum=1
363+
A: busnum=1\n
364364
A: configuration=
365365
H: descriptors=12010002090000406B1D020010030302010109021900010100E0000904000001090000000705810304000C
366366
A: dev=189:0
367-
A: devnum=1
367+
A: devnum=1\n
368368
A: devpath=0
369369
L: driver=../../../../bus/usb/drivers/usb
370370
A: idProduct=0002

‎devices/mobiles/sony-xperia-mini-pro.umockdev‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -38,11 +38,11 @@ A: bNumConfigurations=1
3838
A: bNumInterfaces= 1
3939
A: bcdDevice=0226
4040
A: bmAttributes=c0
41-
A: busnum=1
41+
A: busnum=1\n
4242
A: configuration=
4343
H: descriptors=1201000200000040CE0F660126020203040109022700010100C0FA0904000003FFFF00050705810200020007050202000200070582031C0006
4444
A: dev=189:23
45-
A: devnum=24
45+
A: devnum=24\n
4646
A: devpath=1.5.2.4
4747
A: idProduct=0166
4848
A: idVendor=0fce
@@ -91,11 +91,11 @@ A: bNumConfigurations=1
9191
A: bNumInterfaces= 1
9292
A: bcdDevice=0100
9393
A: bmAttributes=e0
94-
A: busnum=1
94+
A: busnum=1\n
9595
A: configuration=
9696
H: descriptors=12010002090001400904580000010102000109021900010100E0320904000001090000000705810301000C
9797
A: dev=189:19
98-
A: devnum=20
98+
A: devnum=20\n
9999
A: devpath=1.5.2
100100
A: idProduct=0058
101101
A: idVendor=0409
@@ -143,11 +143,11 @@ A: bNumConfigurations=1
143143
A: bNumInterfaces= 1
144144
A: bcdDevice=0001
145145
A: bmAttributes=e0
146-
A: busnum=1
146+
A: busnum=1\n
147147
A: configuration=
148148
H: descriptors=1201000209000240EF17051001000000000109022900010100E0010904000001090001000705810301000C0904000101090002000705810301000C
149149
A: dev=189:10
150-
A: devnum=11
150+
A: devnum=11\n
151151
A: devpath=1.5
152152
A: idProduct=1005
153153
A: idVendor=17ef
@@ -193,11 +193,11 @@ A: bNumConfigurations=1
193193
A: bNumInterfaces= 1
194194
A: bcdDevice=0000
195195
A: bmAttributes=e0
196-
A: busnum=1
196+
A: busnum=1\n
197197
A: configuration=
198198
H: descriptors=12010002090001408780200000000000000109021900010100E0000904000001090000000705810301000C
199199
A: dev=189:1
200-
A: devnum=2
200+
A: devnum=2\n
201201
A: devpath=1
202202
A: idProduct=0020
203203
A: idVendor=8087
@@ -245,11 +245,11 @@ A: bNumConfigurations=1
245245
A: bNumInterfaces= 1
246246
A: bcdDevice=0308
247247
A: bmAttributes=e0
248-
A: busnum=1
248+
A: busnum=1\n
249249
A: configuration=
250250
H: descriptors=12010002090000406B1D020008030302010109021900010100E0000904000001090000000705810304000C
251251
A: dev=189:0
252-
A: devnum=1
252+
A: devnum=1\n
253253
A: devpath=0
254254
A: idProduct=0002
255255
A: idVendor=1d6b

‎src/umockdev-record.vala‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@ write_attr(string name, uint8[] val)
130130
// check if it's text or binary
131131
string strval = (string) val;
132132
if (val.length == strval.length && strval.validate())
133-
stdout.printf("A: %s=%s", name, strval.chomp().escape(""));
133+
stdout.printf("A: %s=%s", name, strval.escape(""));
134134
else
135135
stdout.printf("H: %s=%s", name, format_hex(val));
136136
stdout.putc('\n');

‎tests/test-umockdev.c‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1114,7 +1114,7 @@ t_testbed_usb_lsusb(UMockdevTestbedFixture * fixture, gconstpointer data)
11141114
g_autofree gchar *syspath = umockdev_testbed_add_device(
11151115
fixture->testbed, "usb", "usb1", NULL,
11161116
/* attributes */
1117-
"busnum", "1", "devnum", "1", "speed", "480",
1117+
"busnum", "1\n", "devnum", "1\n", "speed", "480",
11181118
"bConfigurationValue", "1", NULL,
11191119
/* properties */
11201120
"DEVTYPE", "usb_device", "DEVNAME", "/dev/bus/usb/usb1", NULL);

0 commit comments

Comments
 (0)