Compare commits
No commits in common. "c8" and "imports/c8-beta/rasdaemon-0.6.1-12.el8" have entirely different histories.
c8
...
imports/c8
@ -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)
|
|
@ -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)
|
|
@ -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;
|
|
||||||
}
|
|
@ -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]
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user