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
|
||||
Version: 2.2.9
|
||||
Release: 1%{?dist}
|
||||
Release: 2%{?dist}
|
||||
Summary: VPD Database access library for lsvpd
|
||||
|
||||
Group: System Environment/Libraries
|
||||
License: LGPLv2+
|
||||
URL: https://github.com/power-ras/%{name}/releases
|
||||
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: sqlite-devel zlib-devel libstdc++-devel libtool
|
||||
@ -20,15 +21,15 @@ created by vpdupdate in the lsvpd package.
|
||||
|
||||
%package devel
|
||||
Summary: Header files for libvpd
|
||||
Group: Development/Libraries
|
||||
Requires: %{name} = %{version}-%{release} sqlite-devel pkgconfig
|
||||
%description devel
|
||||
Contains header files for building with libvpd.
|
||||
|
||||
%prep
|
||||
%setup -q
|
||||
%autosetup -p1
|
||||
|
||||
%build
|
||||
export CXXFLAGS="-std=c++14 $RPM_OPT_FLAGS"
|
||||
./bootstrap.sh
|
||||
%configure --disable-static
|
||||
make %{?_smp_mflags}
|
||||
@ -57,22 +58,59 @@ find %{buildroot} -type f -name "*.la" -delete
|
||||
%{_libdir}/pkgconfig/libvpd_cxx-2.pc
|
||||
|
||||
%changelog
|
||||
* Fri May 13 2022 Than Ngo <than@redhat.com> - 2.2.9-1
|
||||
- Resolves: #2051316, rebase to 2.2.9
|
||||
* Tue Jan 30 2024 Than Ngo <than@redhat.com> - 2.2.9-2
|
||||
- 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
|
||||
- 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
|
||||
- Resolves: #1664098, rebase to 2.2.7
|
||||
* Tue Oct 27 2020 Jeff Law <law@redhat.com> - 2.2.7-3
|
||||
- Force C++14 as this code is not C++17 ready
|
||||
|
||||
* Wed Jun 05 2019 Than Ngo <than@redhat.com> - 2.2.6-2
|
||||
- Reolves: #1715728, rebuild to get libvpd-devel added to the Builder repo
|
||||
* Tue Jul 28 2020 Fedora Release Engineering <releng@fedoraproject.org> - 2.2.7-2
|
||||
- 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
|
||||
- Rebase to 2.2.6
|
||||
- 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
|
||||
- Rebuilt for https://fedoraproject.org/wiki/Fedora_27_Binutils_Mass_Rebuild
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user