Hello community, here is the log from the commit of package i2c-tools for openSUSE:Factory checked in at 2017-07-12 19:30:17 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/i2c-tools (Old) and /work/SRC/openSUSE:Factory/.i2c-tools.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "i2c-tools" Wed Jul 12 19:30:17 2017 rev:31 rq:508579 version:3.1.2 Changes: -------- --- /work/SRC/openSUSE:Factory/i2c-tools/i2c-tools.changes 2015-12-13 09:37:26.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.i2c-tools.new/i2c-tools.changes 2017-07-12 19:30:17.316765993 +0200 @@ -1,0 +2,18 @@ +Thu Jul 6 10:57:08 UTC 2017 - jdelv...@suse.com + +- decode-dimms-correctly-check-for-out-of-bounds-vendor-id.patch: + Fix recommended by upstream. + +------------------------------------------------------------------- +Thu Jul 6 10:45:42 UTC 2017 - jdelv...@suse.com + +- Update the URL. +- Remove authors list from package description, it does not belong + there. + +------------------------------------------------------------------- +Thu Jul 6 08:16:44 UTC 2017 - jdelv...@suse.com + +- Point to the new download location. + +------------------------------------------------------------------- Old: ---- i2c-tools-3.1.2.tar.bz2 i2c-tools-3.1.2.tar.bz2.sig New: ---- decode-dimms-correctly-check-for-out-of-bounds-vendor-id.patch i2c-tools-3.1.2.tar.sign i2c-tools-3.1.2.tar.xz ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ i2c-tools.spec ++++++ --- /var/tmp/diff_new_pack.InI53q/_old 2017-07-12 19:30:17.908682477 +0200 +++ /var/tmp/diff_new_pack.InI53q/_new 2017-07-12 19:30:17.908682477 +0200 @@ -1,7 +1,7 @@ # # spec file for package i2c-tools # -# Copyright (c) 2015 SUSE LINUX GmbH, Nuernberg, Germany. +# Copyright (c) 2017 SUSE LINUX GmbH, Nuernberg, Germany. # # All modifications and additions to the file contributed by third parties # remain the property of their copyright owners, unless otherwise agreed @@ -17,7 +17,6 @@ Name: i2c-tools -Url: http://www.lm-sensors.org/wiki/I2CTools Version: 3.1.2 Release: 0 Summary: A heterogeneous set of I2C tools for Linux @@ -25,15 +24,14 @@ Group: Development/Tools/Other Requires: udev Recommends: modules -# lm-sensors.org is down for several months now -#Source0: http://dl.lm-sensors.org/%{name}/releases/%{name}-%{version}.tar.bz2 -#Source1: http://dl.lm-sensors.org/%{name}/releases/%{name}-%{version}.tar.bz2.sig -Source0: %{name}-%{version}.tar.bz2 -Source1: %{name}-%{version}.tar.bz2.sig +Url: https://i2c.wiki.kernel.org/index.php/I2C_Tools +Source0: https://www.kernel.org/pub/software/utils/i2c-tools/%{name}-%{version}.tar.xz +Source1: https://www.kernel.org/pub/software/utils/i2c-tools/%{name}-%{version}.tar.sign Patch90: i2c-tools-hackweek-9-dependencies.diff Patch91: i2c-tools-hackweek-9-improve-DDR3-support.diff Patch92: i2c-tools-r6053-Add-smbus-header.patch Patch93: i2c-tools-r6048-Move-SMBus-helper-functions.patch +Patch94: decode-dimms-correctly-check-for-out-of-bounds-vendor-id.patch BuildRoot: %{_tmppath}/%{name}-%{version}-build ExcludeArch: s390 s390x @@ -42,21 +40,13 @@ probing tool, a chip dumper, register-level access helpers, EEPROM decoding scripts, and more. - - -Authors: --------- - Frodo Looijaard - Philip Edelbrock - Mark Studebaker - Jean Delvare <kh...@linux-fr.org> - %prep %setup -q %patch90 -p1 %patch91 -p1 %patch92 -p1 %patch93 -p1 +%patch94 -p1 %build make %{?_smp_mflags} CFLAGS="$RPM_OPT_FLAGS" CC="%{__cc}" ++++++ decode-dimms-correctly-check-for-out-of-bounds-vendor-id.patch ++++++ From: Jean Delvare <jdelv...@suse.de> Date: Thu, 23 Jun 2016 18:59:07 +0200 Subject: decode-dimms: Correctly check for out-of-bounds vendor ID Upstream: yes Git-commit: 2b4135907c1aa27edeb397e62b67386ff5d7d3d5 --- eeprom/decode-dimms | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) --- i2c-tools-3.1.2.orig/eeprom/decode-dimms 2017-07-06 12:55:49.998319548 +0200 +++ i2c-tools-3.1.2/eeprom/decode-dimms 2017-07-06 12:55:51.398337567 +0200 @@ -343,9 +343,11 @@ sub manufacturer_ddr3($$) { my ($count, $code) = @_; return "Invalid" if parity($count) != 1; - return "Invalid" if parity($code) != 1; - return (($code & 0x7F) - 1 > $vendors[$count & 0x7F]) ? "Unknown" : - $vendors[$count & 0x7F][($code & 0x7F) - 1]; + return "Invalid" if parity($code) != 1 + or ($code & 0x7F) == 0; + return "Unknown" if ($count & 0x7F) >= @vendors + or ($code & 0x7F) - 1 >= @{$vendors[$count & 0x7F]}; + return $vendors[$count & 0x7F][($code & 0x7F) - 1]; } sub manufacturer(@)