Compare commits
	
		
			No commits in common. "c8" and "c9-beta" have entirely different histories.
		
	
	
		
	
		
							
								
								
									
										40
									
								
								SOURCES/libvpd-prevent-adding-duplicate-entries.patch
									
									
									
									
									
										Normal file
									
								
							
							
						
						
									
										40
									
								
								SOURCES/libvpd-prevent-adding-duplicate-entries.patch
									
									
									
									
									
										Normal file
									
								
							| @ -0,0 +1,40 @@ | |||||||
|  | commit 34ab92ebc818bc5b356f1d297868512225493eca | ||||||
|  | Author: Sathvika Vasireddy <sv@linux.ibm.com> | ||||||
|  | Date:   Thu Nov 9 02:37:36 2023 +0530 | ||||||
|  | 
 | ||||||
|  |     libvpd: Prevent adding duplicate entries in addDeviceSpecific() function | ||||||
|  |      | ||||||
|  |     Currently, addDeviceSpecific() adds new 'DataItem' to the | ||||||
|  |     'mDeviceSpecific' vector without checking for duplicates. This | ||||||
|  |     vector can be further used by the front end tools to list vpd | ||||||
|  |     information. If this vector has duplicate entries, then vpd | ||||||
|  |     information for a specific device is listed more than once. | ||||||
|  |      | ||||||
|  |     To fix this, prevent duplicate entries with the same 'ac' value | ||||||
|  |     from being stored in the vector. Check if a 'DataItem' with | ||||||
|  |     the same ac value already exists in the vector, and add the new | ||||||
|  |     object to the vector only if no matching item is found. | ||||||
|  |      | ||||||
|  |     Signed-off-by: Sathvika Vasireddy <sv@linux.ibm.com> | ||||||
|  |     Signed-off-by: Mahesh Salgaonkar <mahesh@linux.ibm.com> | ||||||
|  | 
 | ||||||
|  | diff --git a/src/component.cpp b/src/component.cpp
 | ||||||
|  | index 44208ac..a945fc1 100644
 | ||||||
|  | --- a/src/component.cpp
 | ||||||
|  | +++ b/src/component.cpp
 | ||||||
|  | @@ -937,7 +937,6 @@ lderr:
 | ||||||
|  |  		d->ac = ac; | ||||||
|  |  		d->humanName = humanName; | ||||||
|  |  		d->setValue( val, lvl, __FILE__, __LINE__ ); | ||||||
|  | -#if 0
 | ||||||
|  |   | ||||||
|  |  		/* | ||||||
|  |  		 * There can be multiple records with the 'CL' key. | ||||||
|  | @@ -952,7 +951,6 @@ lderr:
 | ||||||
|  |  			if( (*i)->ac == ac ) | ||||||
|  |  				return; //Failed to add - already present | ||||||
|  |  		} | ||||||
|  | -#endif
 | ||||||
|  |   | ||||||
|  |  		mDeviceSpecific.push_back( d ); | ||||||
|  |  	} | ||||||
| @ -1,12 +1,13 @@ | |||||||
| Name:		libvpd | Name:		libvpd | ||||||
| Version:	2.2.9 | Version:	2.2.9 | ||||||
| Release:	1%{?dist} | Release:	2%{?dist} | ||||||
| Summary:	VPD Database access library for lsvpd | Summary:	VPD Database access library for lsvpd | ||||||
| 
 | 
 | ||||||
| Group:		System Environment/Libraries |  | ||||||
| License:	LGPLv2+ | License:	LGPLv2+ | ||||||
| URL:		https://github.com/power-ras/%{name}/releases | URL:		https://github.com/power-ras/%{name}/releases | ||||||
| Source:		https://github.com/power-ras/%{name}/archive/v%{version}/%{name}-%{version}.tar.gz | Source:		https://github.com/power-ras/%{name}/archive/v%{version}/%{name}-%{version}.tar.gz | ||||||
|  | # Prevent adding duplicate entries | ||||||
|  | Patch0: libvpd-prevent-adding-duplicate-entries.patch | ||||||
| 
 | 
 | ||||||
| BuildRequires:	gcc-c++ | BuildRequires:	gcc-c++ | ||||||
| BuildRequires:	sqlite-devel zlib-devel libstdc++-devel libtool | BuildRequires:	sqlite-devel zlib-devel libstdc++-devel libtool | ||||||
| @ -20,15 +21,15 @@ created by vpdupdate in the lsvpd package. | |||||||
| 
 | 
 | ||||||
| %package devel | %package devel | ||||||
| Summary:	Header files for libvpd | Summary:	Header files for libvpd | ||||||
| Group:		Development/Libraries |  | ||||||
| Requires:	%{name} = %{version}-%{release} sqlite-devel pkgconfig | Requires:	%{name} = %{version}-%{release} sqlite-devel pkgconfig | ||||||
| %description devel | %description devel | ||||||
| Contains header files for building with libvpd. | Contains header files for building with libvpd. | ||||||
| 
 | 
 | ||||||
| %prep | %prep | ||||||
| %setup -q | %autosetup -p1 | ||||||
| 
 | 
 | ||||||
| %build | %build | ||||||
|  | export CXXFLAGS="-std=c++14 $RPM_OPT_FLAGS" | ||||||
| ./bootstrap.sh | ./bootstrap.sh | ||||||
| %configure --disable-static | %configure --disable-static | ||||||
| make %{?_smp_mflags} | make %{?_smp_mflags} | ||||||
| @ -57,22 +58,59 @@ find %{buildroot} -type f -name "*.la" -delete | |||||||
| %{_libdir}/pkgconfig/libvpd_cxx-2.pc | %{_libdir}/pkgconfig/libvpd_cxx-2.pc | ||||||
| 
 | 
 | ||||||
| %changelog | %changelog | ||||||
| * Fri May 13 2022 Than Ngo <than@redhat.com> - 2.2.9-1 | * Tue Jan 30 2024 Than Ngo <than@redhat.com> - 2.2.9-2 | ||||||
| - Resolves: #2051316, rebase to 2.2.9 | - Resolves: RHEL-23015, prevent adding duplicate entries | ||||||
|  | 
 | ||||||
|  | * Fri May 06 2022 Than Ngo <than@redhat.com> - 2.2.9-1 | ||||||
|  | - Resolves: #2051288, rebase to 2.2.9 | ||||||
|  | 
 | ||||||
|  | * Mon Aug 09 2021 Mohan Boddu <mboddu@redhat.com> - 2.2.8-4 | ||||||
|  | - Rebuilt for IMA sigs, glibc 2.34, aarch64 flags | ||||||
|  |   Related: rhbz#1991688 | ||||||
|  | 
 | ||||||
|  | * Fri Apr 16 2021 Mohan Boddu <mboddu@redhat.com> - 2.2.8-3 | ||||||
|  | - Rebuilt for RHEL 9 BETA on Apr 15th 2021. Related: rhbz#1947937 | ||||||
|  | 
 | ||||||
|  | * Tue Jan 26 2021 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.8-2 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_34_Mass_Rebuild | ||||||
| 
 | 
 | ||||||
| * Mon Nov 09 2020 Than Ngo <than@redhat.com> - 2.2.8-1 | * Mon Nov 09 2020 Than Ngo <than@redhat.com> - 2.2.8-1 | ||||||
| - Resolves: #1844429, rebase to 2.2.8 | - rebase to 2.2.8 | ||||||
| 
 | 
 | ||||||
| * Mon Apr 06 2020 Than Ngo <than@redhat.com> - 2.2.7-1 | * Tue Oct 27 2020 Jeff Law <law@redhat.com> - 2.2.7-3 | ||||||
| - Resolves: #1664098, rebase to 2.2.7 | - Force C++14 as this code is not C++17 ready | ||||||
| 
 | 
 | ||||||
| * Wed Jun 05 2019 Than Ngo <than@redhat.com> - 2.2.6-2 | * Tue Jul 28 2020 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.7-2 | ||||||
| - Reolves: #1715728, rebuild to get libvpd-devel added to the Builder repo | - Rebuilt for https://fedoraproject.org/wiki/Fedora_33_Mass_Rebuild | ||||||
|  | 
 | ||||||
|  | * Thu Jul 09 2020 Than Ngo <than@redhat.com> - 2.2.7-1 | ||||||
|  | - rebase to 2.2.7 | ||||||
|  | 
 | ||||||
|  | * Wed Jan 29 2020 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.6-7 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_32_Mass_Rebuild | ||||||
|  | 
 | ||||||
|  | * Mon Dec 02 2019 Than Ngo <than@redhat.com> - 2.2.6-6 | ||||||
|  | - Update Url and Source | ||||||
|  | 
 | ||||||
|  | * Thu Jul 25 2019 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.6-5 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_31_Mass_Rebuild | ||||||
|  | 
 | ||||||
|  | * Fri Feb 01 2019 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.6-4 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_30_Mass_Rebuild | ||||||
|  | 
 | ||||||
|  | * Fri Jul 20 2018 Sinny Kumari <sinnykumari@fedoraproject.org> - 2.2.6-3 | ||||||
|  | - Add gcc-c++ as BuildRequires | ||||||
|  | 
 | ||||||
|  | * Fri Jul 13 2018 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.6-2 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_29_Mass_Rebuild | ||||||
| 
 | 
 | ||||||
| * Fri Jul 06 2018 Sinny Kumari <sinnykumari@fedoraproject.org> - 2.2.6-1 | * Fri Jul 06 2018 Sinny Kumari <sinnykumari@fedoraproject.org> - 2.2.6-1 | ||||||
| - Rebase to 2.2.6 | - Rebase to 2.2.6 | ||||||
| - run.vpdupdate is now created in /run instead of /var/lib/lsvpd/ | - run.vpdupdate is now created in /run instead of /var/lib/lsvpd/ | ||||||
| 
 | 
 | ||||||
|  | * Wed Feb 07 2018 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.5-8 | ||||||
|  | - Rebuilt for https://fedoraproject.org/wiki/Fedora_28_Mass_Rebuild | ||||||
|  | 
 | ||||||
| * Thu Aug 03 2017 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.5-7 | * Thu Aug 03 2017 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.5-7 | ||||||
| - Rebuilt for https://fedoraproject.org/wiki/Fedora_27_Binutils_Mass_Rebuild | - Rebuilt for https://fedoraproject.org/wiki/Fedora_27_Binutils_Mass_Rebuild | ||||||
| 
 | 
 | ||||||
|  | |||||||
		Loading…
	
		Reference in New Issue
	
	Block a user