diff options
| author | Nathan Scott <nathans@redhat.com> | 2023-09-01 09:34:46 +1000 |
|---|---|---|
| committer | Nathan Scott <nathans@redhat.com> | 2023-09-04 08:59:41 +1000 |
| commit | 66c0b19ed6fec2afd4ed381665274fa83c67660e (patch) | |
| tree | e8420e98aa4d5fb633cbb432babfa86d3dcdaf37 /solaris | |
| parent | 9392e1d596dd2bed165c8cb522a0c9fd5fede66d (diff) | |
Correct pid_t type return from Platform_getMaxPid function
Coverity scanning shows we end up passing an integer into the
Row_setPidColumnWidth routine which requires a pid_t - update
each platform to return the correct type (and never return -1
as a failure code, this was being ignored).
Diffstat (limited to 'solaris')
| -rw-r--r-- | solaris/Platform.c | 2 | ||||
| -rw-r--r-- | solaris/Platform.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/solaris/Platform.c b/solaris/Platform.c index f14ae2e9..5117a448 100644 --- a/solaris/Platform.c +++ b/solaris/Platform.c @@ -175,7 +175,7 @@ void Platform_getLoadAverage(double* one, double* five, double* fifteen) { *fifteen = plat_loadavg[LOADAVG_15MIN]; } -int Platform_getMaxPid(void) { +pid_t Platform_getMaxPid(void) { int vproc = 32778; // Reasonable Solaris default kstat_ctl_t* kc = kstat_open(); diff --git a/solaris/Platform.h b/solaris/Platform.h index 00bc5832..96feeebe 100644 --- a/solaris/Platform.h +++ b/solaris/Platform.h @@ -72,7 +72,7 @@ int Platform_getUptime(void); void Platform_getLoadAverage(double* one, double* five, double* fifteen); -int Platform_getMaxPid(void); +pid_t Platform_getMaxPid(void); double Platform_setCPUValues(Meter* this, unsigned int cpu); |