Module Name: src Committed By: jakllsch Date: Fri Jan 1 22:59:12 UTC 2016
Modified Files: src/lib/libusbhid: data.c src/tests/lib/libusbhid: t_usbhid.c Log Message: Fix hid_get_data() for negative or 32-bit report data. To generate a diff of this commit: cvs rdiff -u -r1.6 -r1.7 src/lib/libusbhid/data.c cvs rdiff -u -r1.2 -r1.3 src/tests/lib/libusbhid/t_usbhid.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/lib/libusbhid/data.c diff -u src/lib/libusbhid/data.c:1.6 src/lib/libusbhid/data.c:1.7 --- src/lib/libusbhid/data.c:1.6 Wed May 12 18:28:20 2010 +++ src/lib/libusbhid/data.c Fri Jan 1 22:59:12 2016 @@ -1,4 +1,4 @@ -/* $NetBSD: data.c,v 1.6 2010/05/12 18:28:20 plunky Exp $ */ +/* $NetBSD: data.c,v 1.7 2016/01/01 22:59:12 jakllsch Exp $ */ /* * Copyright (c) 1999 Lennart Augustsson <augus...@netbsd.org> @@ -27,7 +27,7 @@ */ #include <sys/cdefs.h> -__RCSID("$NetBSD: data.c,v 1.6 2010/05/12 18:28:20 plunky Exp $"); +__RCSID("$NetBSD: data.c,v 1.7 2016/01/01 22:59:12 jakllsch Exp $"); #include <assert.h> #include <stdlib.h> @@ -57,10 +57,12 @@ hid_get_data(const void *p, const hid_it for (i = 0; i < end; i++) data |= buf[offs + i] << (i*8); data >>= hpos % 8; - data &= (1 << hsize) - 1; - if (h->logical_minimum < 0 && (data & (1<<(hsize-1)))) { - /* Need to sign extend */ - data |= 0xffffffff & ~((1<<hsize)-1); + if (hsize < 32) { + data &= (1 << hsize) - 1; + if (h->logical_minimum < 0 && (data & (1<<(hsize-1)))) { + /* Need to sign extend */ + data |= 0xffffffff & ~((1<<hsize)-1); + } } return (int)(data); } Index: src/tests/lib/libusbhid/t_usbhid.c diff -u src/tests/lib/libusbhid/t_usbhid.c:1.2 src/tests/lib/libusbhid/t_usbhid.c:1.3 --- src/tests/lib/libusbhid/t_usbhid.c:1.2 Fri Jan 1 22:47:34 2016 +++ src/tests/lib/libusbhid/t_usbhid.c Fri Jan 1 22:59:12 2016 @@ -1,4 +1,4 @@ -/* $NetBSD: t_usbhid.c,v 1.2 2016/01/01 22:47:34 jakllsch Exp $ */ +/* $NetBSD: t_usbhid.c,v 1.3 2016/01/01 22:59:12 jakllsch Exp $ */ /* * Copyright (c) 2016 Jonathan A. Kollasch @@ -27,7 +27,7 @@ */ #include <sys/cdefs.h> -__RCSID("$NetBSD: t_usbhid.c,v 1.2 2016/01/01 22:47:34 jakllsch Exp $"); +__RCSID("$NetBSD: t_usbhid.c,v 1.3 2016/01/01 22:59:12 jakllsch Exp $"); #include <atf-c.h> @@ -326,9 +326,6 @@ ATF_TC_BODY(check_hid_get_data, tc) int32_t data; uint32_t udat; - atf_tc_expect_fail("all negative data doesn't work, " - "all 32-bit data doesn't work"); - ATF_REQUIRE((hrd = hid_use_report_desc( range_test_report_descriptor, __arraycount(range_test_report_descriptor))) != NULL);