Compare commits

...

No commits in common. "c8" and "imports/c8-beta/rasdaemon-0.6.1-12.el8" have entirely different histories.

4 changed files with 1 additions and 133 deletions

View File

@ -1,71 +0,0 @@
commit 899fcc2cf21c86b5462c8f4441cd9c92b3d75f7d
Author: Aristeu Rozanski <arozansk@redhat.com>
Date: Thu Jan 19 08:45:57 2023 -0500
rasdaemon: ras-report: fix possible but unlikely file descriptor leak
Found with covscan.
Signed-off-by: Aristeu Rozanski <arozansk@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
---
ras-report.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
--- rasdaemon-0.6.1.orig/ras-report.c 2023-01-23 11:36:20.972368760 -0500
+++ rasdaemon-0.6.1/ras-report.c 2023-01-23 11:36:23.236343267 -0500
@@ -374,7 +374,7 @@ if(rc < 0){
mc_fail:
- if(sockfd > 0){
+ if(sockfd >= 0){
close(sockfd);
}
@@ -424,7 +424,7 @@ if(rc < 0){
aer_fail:
- if(sockfd > 0){
+ if(sockfd >= 0){
close(sockfd);
}
@@ -473,7 +473,7 @@ rc = 0;
non_standard_fail:
- if(sockfd > 0){
+ if(sockfd >= 0){
close(sockfd);
}
@@ -518,7 +518,7 @@ rc = 0;
arm_fail:
- if(sockfd > 0){
+ if(sockfd >= 0){
close(sockfd);
}
@@ -564,7 +564,7 @@ if(rc < 0){
mce_fail:
- if(sockfd > 0){
+ if(sockfd >= 0){
close(sockfd);
}
@@ -609,7 +609,7 @@ if (rc < 0)
done = 1;
mf_fail:
- if (sockfd > 0)
+ if (sockfd >= 0)
close(sockfd);
if (done)

View File

@ -1,28 +0,0 @@
commit ce33041e0abfa20054ff5d6874ffbd1ab592558d
Author: Aristeu Rozanski <arozansk@redhat.com>
Date: Thu Jan 19 08:45:57 2023 -0500
rasdaemon: ras-memory-failure-handler: handle localtime() failure correctly
We could just have an empty string but keeping the format could prevent
issues if someone is actually parsing this.
Found with covscan.
v2: fixed the timestamp as pointed by Robert Elliott
Signed-off-by: Aristeu Rozanski <arozansk@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
diff --git a/ras-memory-failure-handler.c b/ras-memory-failure-handler.c
index 9941e68..1951456 100644
--- a/ras-memory-failure-handler.c
+++ b/ras-memory-failure-handler.c
@@ -148,6 +148,8 @@ int ras_memory_failure_event_handler(struct trace_seq *s,
if (tm)
strftime(ev.timestamp, sizeof(ev.timestamp),
"%Y-%m-%d %H:%M:%S %z", tm);
+ else
+ strncpy(ev.timestamp, "1970-01-01 00:00:00 +0000", sizeof(ev.timestamp));
trace_seq_printf(s, "%s ", ev.timestamp);
if (pevent_get_field_val(s, event, "pfn", record, &val, 1) < 0)

View File

@ -1,24 +0,0 @@
commit e8b97ec14a11764fedfea50bd4d96ddda43c7fc1
Author: Aristeu Rozanski <arozansk@redhat.com>
Date: Thu Jan 19 08:45:57 2023 -0500
rasdaemon: mce-amd-smca: properly limit bank types
Found with covscan.
Signed-off-by: Aristeu Rozanski <arozansk@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
diff --git a/mce-amd-smca.c b/mce-amd-smca.c
index f3379fc..27ca8aa 100644
--- a/mce-amd-smca.c
+++ b/mce-amd-smca.c
@@ -584,7 +584,7 @@ static void decode_smca_error(struct mce_event *e)
return;
}
- if (bank_type >= MAX_NR_BANKS) {
+ if (bank_type >= N_SMCA_BANK_TYPES) {
strcpy(e->mcastatus_msg, "Don't know how to decode this bank");
return;
}

View File

@ -1,6 +1,6 @@
Name: rasdaemon Name: rasdaemon
Version: 0.6.1 Version: 0.6.1
Release: 13%{?dist} Release: 12%{?dist}
Summary: Utility to receive RAS error tracings Summary: Utility to receive RAS error tracings
Group: Applications/System Group: Applications/System
License: GPLv2 License: GPLv2
@ -44,9 +44,6 @@ Patch17: 16d929b024c31d54a7f8a72eab094376c7be27f5.patch
Patch18: b497a3d6a39d402c41065e9284d49114b97e3bfe.patch Patch18: b497a3d6a39d402c41065e9284d49114b97e3bfe.patch
Patch19: ce6e7864f11f709c4f803828fbc8e507d115d03b.patch Patch19: ce6e7864f11f709c4f803828fbc8e507d115d03b.patch
Patch20: a8c776ed94f68ae31d7b5f74e19545698898c13c.patch Patch20: a8c776ed94f68ae31d7b5f74e19545698898c13c.patch
Patch21: 899fcc2cf21c86b5462c8f4441cd9c92b3d75f7d.patch
Patch22: e8b97ec14a11764fedfea50bd4d96ddda43c7fc1.patch
Patch23: ce33041e0abfa20054ff5d6874ffbd1ab592558d.patch
%description %description
%{name} is a RAS (Reliability, Availability and Serviceability) logging tool. %{name} is a RAS (Reliability, Availability and Serviceability) logging tool.
@ -80,9 +77,6 @@ an utility for reporting current error counts from the EDAC sysfs files.
%patch18 -p1 %patch18 -p1
%patch19 -p1 %patch19 -p1
%patch20 -p1 %patch20 -p1
%patch21 -p1
%patch22 -p1
%patch23 -p1
# The tarball is locked in time the first time aclocal was ran and will keep # The tarball is locked in time the first time aclocal was ran and will keep
# requiring an older version of automake # requiring an older version of automake
@ -113,9 +107,6 @@ rm INSTALL %{buildroot}/usr/include/*.h
%{_sysconfdir}/ras/dimm_labels.d %{_sysconfdir}/ras/dimm_labels.d
%changelog %changelog
* Mon Jan 23 2023 Aristeu Rozanski <aris@redhat.com> 0.6.1-13
- Fixing covscan issues [2073516]
* Tue Oct 12 2021 Aristeu Rozanski <aris@redhat.com> 0.6.1-12 * Tue Oct 12 2021 Aristeu Rozanski <aris@redhat.com> 0.6.1-12
- Adding missing bits from b497a3d6a39d402c41065e9284d49114b97e3bfe [1923254] - Adding missing bits from b497a3d6a39d402c41065e9284d49114b97e3bfe [1923254]