Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Split large functions into smaller ones

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

simtest: Fix do_test script

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

svg_stats.c: Ignore negative values for fields position

Negative values in g_fields[] array are now ignored in save_extrema()
function.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

systest.c: Declare fs data with const

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Specify field width when using sscanf() function

This is to avoid possible crashes with huge input data.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sa_common.c: Don't use (void *) pointer in calculation

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Reduce variables scope when relevant

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Don't assign values that are never used

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Free pointer if realloc() fails

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Declare parameters with 'const' when possible

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Remove conditions which are always true

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

iostat: Clarify calculation precedence for '+' and '?'

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'lzaoral-fix-dec-opt'

Merge branch 'fix-dec-opt' of https://github.com/lzaoral/sysstat into lzaoral-fix-dec-opt

SIMTEST: Tests which are skipped touch file tests/SKIPPED

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Small update to Makefile.in file

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update do_test script for sysstat simulation test environment

Tests are now run from the script itself instead of the Makefile.
Several options have been added ("cont", "ignore"...)

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Tools that take --dec=X option should only accept digits

Right now the argument of --dec is passed to atoi(3) which returns 0
on conversion error.  Therefore, --dec=A was not rejected and was
equivalent to --dec=0 by mistake.

Signed-off-by: Lukáš Zaoral <lzaoral@redhat.com>

sadf_misc.c: Fix indentation in code

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Don't check if unsigned expressions are less than zero

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Fix types used in format strings

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Fix typo in sar's manual page

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Stop when invalid data are read in record's header

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Defaut time value has changed.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

SIMTEST: Update default unix_time value

Default value for the number of seconds since the Epoch is no longer 0
(01/01/1970) but 1591016000 which corresponds to 06/01/2020.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Add a check on file's records headers data

Make sure that the number of seconds since the Epoch saved in the
records's headers are at least 10-digit numbers.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Restore mode for iconfig script

iconfig script is an executable file.
Also update Makefile so that proper mode is restored when updating
copyright message.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Starting sysstat 12.7.3

configure script updated.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sysstat-12.7.2

sysstat version 12.7.2 final packaging.
Changelog added.
spec and lsm files updated.

This version adds a new activity to sar and sadf: Batteries power
statistics, which can be displayed using option "-m BAT".
Output has also been improved so that healthy metrics values (for all
sysstat commands) will no longer be displayed in "red" (or any other
specific color).
Last, starting and ending times can now be entered as a nuumber of
seconds since the Epoch with sar and sadf's options -s/-e.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update NLS translations

Sync with The Translation Project.
Belarusian translation added.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update year in copyright message

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Remove LGTM links from README file

LGTM.com is now deprecated. Remove corresponding links.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add new non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Rework code related to options -s/-e

Timestamps entered as a number of seconds since the Epoch are now used
as they are and no longer converted to a broken time representation, so
that even month and year are taken into account when current record time
is compared to those timestamps.

The record which is exactly at the ending time is now displayed.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar/sadf: Strengthen tests made on args given to options -s/-e (#350)

Make additional tests on the argument following options -s and -s used
with sar and sadf to make sure it is actually a timestamp before using
it as such.
Previously, in the examples below, xfile (5 chars) and my_xfile (8
chars) were considered as timestamps in parse_timestamp() function,
although they actually are data files names.

sadf -s xfile
or
sadf -s my_xfile

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: Replace time_start/time_end with start_time/end_time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Replace time_start/time_end with start_time/end_time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sar and sadf usage message

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update sar and sadf manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'NatePW-sadf_epoch_time'

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Remove unneeded break after return in switch case

Replace potentially dangerous localtime function

Compare UTC time properly format agnostic

Add epoch type to start and end times

sar: Make sure timestamps are always displayed in local time

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: Fix help message

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: XML: A_PWR_BAT: Fix XSD document

Variation of battery capacity may be negative. So update XSD document to
accept negative values.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: PCP: Fix pmiID number for three USB metrics

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Generate batteries statistics in PCP format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Add f_count_new() function

This function is aimed at creating a list of battery names that will
later be used to create PCP instances.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Use colors to display negative values

Two thresholds are defined: -10.0 and -5.0.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Change field name from "%cap/min" to "cap/min"

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sar: A_PWR_BAT: Change arrow type for status "Not charging"

Use East Arrow symbol (right arrow) instead of South Arrow for status
"Not charging".

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: JSON: A_PWR_BAT: Add missing check

Make sure that status code is not greater than upper limit before using.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in RAW format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Move some definitions from common.h to rd_stats.h

proc and sys files which are used only by sadc are moved to rd_stats.h
file.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in SVG format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in JSON format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in XML format

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Update DTD and XSD documents

Update XML documents to deal with batteries statistics.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: A_PWR_BAT: Display batteries stats in db/ppc formats

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Change column name from "sts" to "status"

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

common.c: Display sign with cprintf_f() function

Update cprintf_f() function so that the sign (+ or -) can be displayed
if requested.
This is intended for values used for batteries capacity variation.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Makefile: Fix dependencies

Files pcp_def_metrics.c and pcp_stats.c depend on sa.h which depends
itself on common.h, rd_stats.h and rd_sensors.h.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Display battery statistics with sar

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Add f_count and f_read functions

Add functions used to count number of batteries installed and to read
batteries statistics.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

A_PWR_BAT: Add new option and basic definitions for sar and sadc

Add new activity A_PWR_BAT.
It will enable the user to monitor battery capacity, variation and
status.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Cosmetic changes (enumerate values in header files when applicable)

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sadf: RAW: Fix extra space when no TZ printed

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge branch 'NatePW-correct_dbpcc_timestamp'

Merge branch 'correct_dbpcc_timestamp' of https://github.com/NatePW/sysstat into NatePW-correct_dbpcc_timestamp

sadc: Add another overflow check

Make sure resulting value won't overflow UINT_MAX before allocating
memory.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update manual pages

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Try to avoid displaying healthy metrics values in "red" (#177)

Percentage values are now displayed in specific colors when they are
greater or lower than a predefined limit depending on the metric's
meaning.
This means that, e.g. CPU %idle value will no longer be printed in red
if the CPU is actually idle.

Specific colors are now used when:
* values are greater than 90% (or lower than 10%),
* values are in the range 75% to 90% (or 25% to 10%).

This is probably still not perfect but should eliminate many questions.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Fix extra space when no TZ printed

Merge pull request #346 from Kwstubbs/Kwstubbs/add-codeql-workflow

Add CodeQL workflow

Add CodeQL workflow

Starting sysstat 12.7.2

configure script updated.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

sysstat-12.7.1

sysstat version 12.7.1 final packaging.
Changelog added.
spec and lsm files updated.

This version adds some small new features, like option -H to mpstat to
handle vCPU physical hotplug, or true time mode support to sadf's SVG
output.
It also fixes several bugs, one of which has been the object of a
security advisory (CVE-2022-39377).

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update NLS translations

Sync with the Translation Project.
Add new Georgian translation.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Make sure values to be compared are unsigned integers

It seems safer to make sure that input values are unsigned int before
casting them to unsigned long long and making the comparison.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests again

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Update non regression tests

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Add more overflow checks

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>

Merge pull request from GHSA-q8r6-g56f-9w7x

Fix size_t overflow in sa_common.c (GHSL-2022-074)

Merge pull request #342 from shaleen-bathla/master

mpstat: return void from rw_mpstat_loop

mpstat: return void from rw_mpstat_loop

Previous commits removed the usage of return value
from rw_mpstat_loop()
Hence change return value from int to void.

Signed-off-by: Shaleen Bathla <shaleen.bathla@oracle.com>

Fix size_t overflow in sa_common.c (GHSL-2022-074)

allocate_structures function located in sa_common.c insufficiently
checks bounds before arithmetic multiplication allowing for an
overflow in the size allocated for the buffer representing system
activities.

This patch checks that the post-multiplied value is not greater than
UINT_MAX.

Signed-off-by: Sebastien <seb@fedora-2.home>

Update non regression tests

...and add a new non regression test for mpstat -H.

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>