proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-11-02 16:25:18 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #10655 from donaldsharp/timers_warning_when_large

Browse Source 
lib: Add a Dev catch for when a timer is set for > 1 year
This commit is contained in:
Sri Mohana Singamsetty 2022-02-25 08:05:45 -08:00 committed by GitHub
commit 787bd3ba6c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/lib_errors.c
View File

@ -68,6 +68,12 @@ static struct log_ref ferr_lib_warn[] = {
.description = "The Event subsystem has detected a file descriptor read/write event without an associated handling function. This is a bug, please collect log data and open an issue.",
.suggestion = "Gather log data and open an Issue",
},
{
.code = EC_LIB_TIMER_TOO_LONG,
.title = "The Event subsystem has detected an internal timer that is scheduled to pop in greater than one year",
.description = "The Event subsystem has detected a timer being started that will pop in a timer that is greater than one year. This is a bug, please collect log data and open an issue.",
.suggestion = "Gather log data and open an Issue",
},
{
.code = EC_LIB_RMAP_RECURSION_LIMIT,
.title = "Reached the Route-Map Recursion Limit",

1
lib/lib_errors.h
View File

@ -48,6 +48,7 @@ enum lib_log_refs {
EC_LIB_SLOW_THREAD_WALL,
EC_LIB_STARVE_THREAD,
EC_LIB_NO_THREAD,
EC_LIB_TIMER_TOO_LONG,
EC_LIB_RMAP_RECURSION_LIMIT,
EC_LIB_BACKUP_CONFIG,
EC_LIB_VRF_LENGTH,

6
lib/thread.c
View File

@ -1052,6 +1052,12 @@ static void _thread_add_timer_timeval(const struct xref_threadsched *xref,
if (thread_timer_list_first(&m->timer) == thread)
AWAKEN(m);
}
#define ONEYEAR2SEC (60 * 60 * 24 * 365)
if (time_relative->tv_sec > ONEYEAR2SEC)
flog_err(
EC_LIB_TIMER_TOO_LONG,
"Timer: %pTHD is created with an expiration that is greater than 1 year",
thread);
}