github: Run the cgclear functional test last

Run the cgclear functional test last since it is destructive to the
cgroup hierarchy.  Note that this test is automatically skipped by
'make check'.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

bootstrap: github: Ignore systemd hierarchy

Ignore the "name=systemd" hierarchy by setting the autoconf
--enable-opaque-hierarchy flag equal to "name=systemd".  This
instructs libcgroup to ignore mounts named "name=systemd" and
thus these mounts aren't added to the cg_mount_table[].

Suggested-by: Michal Koutný <mkoutny@suse.com>
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

config.c: Add cgroup v2 support to cgclear

When cgclear is invoked against a cgroup v2 controller,
do not attempt to unmount the v2 controller.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #30 from drakenclimber/issues/cgset-tests

api.c: Don't fail a recursive write if value isn't dirty

api.c: Don't fail a recursive write if value isn't dirty

When cgroup_modify_cgroup() invokes cgroup_set_values_recursive(),
some of the settings within the cgroup may not be writable.  Avoid
failing the entire write by ignoring write failures on settings that
do not explicitly have the dirty flag set.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

bootstrap: Use main branch of libcgroup-tests

The default branch of libcgroup-tests has been renamed to
main.  Utilize this branch.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #27 from drakenclimber/issues/daemoncodecov

Makefile: Run code coverage on the daemon folder

Makefile: Run code coverage on the daemon folder

Code coverage was not being gathered for src/daemon.
Add code coverage support for this directory.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #26 from drakenclimber/issues/cgget-refactor2

Refactor cgget

cgget: Major refactor

Previously cgget used data structures unlike anywhere else
within libcgroup.  cgget had arrays of strings to hold the
list of controllers, settings, and values.

Update cgget to utilize struct cgroup and struct
cgroup_controller.

This patch is intended to have no functional changes.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

wrapper.c: Add support for empty values in cgroup_add_value_string()

cgroup_add_value_string() is the fundamental building block
for adding a name/value pair to a cgroup_controller struct.
Add support for a NULL value field.

Currently it only supports adding a valid value, but in a
subsequent commit, cgget will utilize this function to build
up a hierarchy of cgroups, controllers, and setting names.
The values are NULL at the time the struct are populated and
will be read from sysfs at a later time.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Expose cgroup_test_subsys_mounted() to users of libcgroup.la

Remove the static qualifier from cgroup_test_subsys_mounted().  As
part of the cgget refactor, cgget will utilize this function.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Expose cgroup_fill_cgc() to users of libcgroup.la

Remove the static qualifier from cgroup_fill_cgc().  As part
of the cgget refactor, cgget will utilize this function.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api: Expose cg_build_path_locked() to users of libcgroup.la

Remove the static qualifier from cg_build_path_locked().  As part
of the cgget refactor, cgget will utilize this function.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #25 from drakenclimber/issues/nocontainer

Run tests outside of a container

github: Log the results of the non-containerized tests

Log the results of the non-containerized tests.  The
non-containerized tests will be automatically invoked
by `make check`.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #23 from drakenclimber/issues/21

api.c: Fix erroneous error handling in cgroup_set_values_recursive()

api.c: Fix erroneous error handling in cgroup_set_values_recursive()

Ignore the positive length returned by asprintf() and only act
upon the error code returned from cg_set_control_value().  Previously
cgroup_set_values_recursive() mishandled the error handling when
the value's dirty flag was set.

Reported-by: Bharani viswas <bharani.viswas@gmail.com>
Fixes: 092e4aeb0416f ("api.c: Refactor duplicate logic in cgroup modify and create")
Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

github: Always display test logs

Always display the test logs in the github actions
GUI.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #19 from drakenclimber/issues/cgsnapshot2

Add support for cgroup v2 cgsnapshot

cgsnapshot: Replace hardcoded "tasks" file

Replace the hardcoded building of the tasks path in
cgsnapshot with a call to cgroup_build_tasks_procs_path().
By calling cgroup_build_tasks_procs_path(), cgsnapshot
now supports both cgroup v1 and cgroup v2.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Reviewed-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: Export cgroup_build_tasks_procs_path()

Make cgroup_build_tasks_procs_path() available to
libcgroup.la library users (i.e. the libcgroup tools).

This change will be used in the next commit so that
cgsnapshot can properly read the tasks/procs file.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Reviewed-by: Dhaval Giani <dhaval.giani@oracle.com>

github: Display ftests.sh.log in the CI GUI

Now that we invoke ftests.sh in the functional tests'
Makefile.am, test-suite.log isn't that useful.  Display
ftests.sh.log in the Github Actions GUI since
ftests.sh.log contains pass/fail info for each test.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #18 from drakenclimber/issues/betterlogs

github: Improve logging and reporting

github: Improve logging and reporting

Display the high-level log, test-suite.log, in
the github work for all jobs.  Also, archive the
functional test logs to facilitate debugging a
failed run.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

README.md: Update the coveralls badge

The coveralls.io badge wasn't consistently updating
after a successful run.  Remove the "?branch=master"
parameter and switch to using a png rather than an
svg.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #17 from drakenclimber/issues/github_actions

Transition from Travis CI to Github Actions

README.md: Transition from Travis CI to Github Actions

Travis CI has now been disabled.  Delete the Travis CI
badge and display the Github Actions continuous integration
workflow badge.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

github: Add continuous integration Github workflow

Add a Github Actions continuous integration workflow.  This
workflow runs the unit tests, the cgroup v1 functional tests,
and the cgroup v2 functional tests.  The code coverage results
from all of the runs are collated at coveralls.io.

Like Travis CI, Github Actions currently does not provide a
distro that natively runs cgroup v2.  (And it also doesn't
expose the kernel boot command line.)  So, like our previous
Travis CI solution, unmount the cpuset controller and enable
it in cgroup v2.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

github: Add a Github Action to setup the libcgroup directory

Add a Github Action to setup the libcgroup directory.
This action installs the required dependencies, run
bootstrap.sh, runs configure, and invokes make.

Since the action is invoked locally (and not via Github
Action's published action mechanism), the repository must
be git cloned prior to running this action.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

travis: Disable Travis CI

Disable the Travis CI build by deleting the
.travis.yml file.  Subsequent commits will transition
to using Github Actions instead.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #16 from drakenclimber/issues/ftests_cgv2

Add cgroup v2 support to the functional tests

travis: Add test job on cgroup v2 Ubuntu 20.04

Add a test job on Ubuntu 20.04.  Prior to running the tests, unmount
the cpuset cgroup controller and enable it in the cgroup v2 unified
hierarchy.  Once each job is complete, it notifies coveralls via a
REST call.

Unfortunately, Travis CI doesn't provide a cgroup v2 system image
and also doesn't allow us to modify the kernel boot parameters.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

travis: Exclude lex.l and parse.y from code coverage

Exclude lex.l and parse.y from code coverage numbers.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #14 from drakenclimber/issues/cgdelete

api.c: Replace hardcoded "/tasks" in delete functions

api.c: Replace hardcoded "/tasks" in delete functions

Replace the hardcoded building of the tasks path in
cg_delete_cgroup_controller() and cgroup_delete_cgroup_ext()
with a call to cgroup_build_tasks_procs_path().  By calling
cgroup_build_tasks_procs_path(), both delete functions now
support cgroup v1 and v2.

cgdelete now supports cgroup v2.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #13 from drakenclimber/issues/cgclassify2

Add cgroup v2 support for cgclassify and cgexec

api.c: Verify if a controller is enabled for a v2 cgroup

Add a function, cgroupv2_controller_enabled(), that checks if the
requested cgroup v2 controller can be used for the specified path.
Again, in cgroup v2, to use a controller, it must be added to the
parent cgroup's cgroup.subtree_control file.

This check is required to ensure that the cgexec and cgclassify
operations will succeed.  And with this commit, cgexec and cgclassify
work on both cgroup v1 and cgroup v2 hierarchies.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add function to read the cgroup.subtree_control file

Add a function, cgroupv2_get_subtree_control(), that verifies
if a controller is enabled in the cgroup.subtree_control file
for the requested path.

This function will be used in the next commit - "api.c: Add function
to check if a controller is enabled for a cgroup v2 cgroup" - to
determine if the requested cgroup v2 controller is enabled for a
cgroup.  This is a required check to ensure that the
cgclassify/cgexec operation will succeed.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add function to build tasks/procs path

Replace the inline logic in multiple functions that concatenates
the path and cgroup v1 tasks file with a function -
cgroup_build_tasks_procs_path() - that works for both cgroup v1
and v2.  In the v1 case, it appends tasks, and in the v2 case, it
appends cgroup.procs.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Revert "DO NOT COMMIT"

This reverts commit 83d6205fc7711818bd2e4bf1890d7d157756182e.

Merge pull request #11 from drakenclimber/issues/cgcreate

Add cgroup v2 support to cgcreate

api.c: Add cgroup v2 support to cgroup_create_cgroup()

This commit adds cgroup v2 support go cgroup_create_cgroup().
If the controller is mounted via cgroup v2, then the
subtree_control file in the parent directory will be updated
to enable the controller in children cgroups.  The tasks
file is only updated for cgroup v1 mounts.

Here's an example of creating a cgroup hierarchy in a single command:
sudo cgcreate -g io:a/b/c/d

The above cgcreate example is equivalent to:
mkdir -p {root_cg}/a/b/c/d
echo +io > {root_cg}/cgroup.subtree_control
echo +io > {root_cg}/a/cgroup.subtree_control
echo +io > {root_cg}/a/b/cgroup.subtree_control
echo +io > {root_cg}/a/b/c/cgroup.subtree_control

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add function to recursively manage the cgroup v2 subtree

This commit adds the function cgroupv2_subtree_control_recursive().
cgroupv2_subtree_control_recursive() creates a cgroup hierarchy with
the subtree_control enabled in each parent cgroup for the requested
cgroup controller.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add function to manage the cgroup v2 subtree_control file

This commit adds the function cgroupv2_subtree_control().
cgroupv2_subtree_control() can be used to enable or disable a
controller in the subtree_control file.  The equivalent command
line is:
echo +{ctrl_name} > {path}/cgroup.subtree_control
echo -{ctrl_name} > {path}/cgroup.subtree_control

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add function to get the cgroup version of a controller

This commit adds cgroup_get_controller_version() which returns
which version (cgroup v1 or cgroup v2) of a controller is mounted.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Refactor tasks file permissions code into its own function

Within cgroup_create_cgroup(), there was a block of code to set the
owner, group, and file permissions on the cgroup v1 tasks file.  This
commit moves that logic into its own function -
cgroup_chown_chmod_tasks().

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Refactor duplicate logic in cgroup modify and create

cgroup_modify_cgroup() and cgroup_create_cgroup() have nearly
identical logic to walk the settings beneath the controller
and update them.  This commit refactors the duplicate logic in
these two functions into one location - cgroup_set_values_recursive().

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

travis: Show the gunit log on failure

When a failure occurs in TravisCI, we currently show only the
test-suite.log from the functional tests.  This commit also displays
the googletest test-suite.log.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

DO NOT COMMIT

This commit points the testing directory at my personal fork

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #8 from kloczek/master

Fix automake bug for parallel makefiles

Paralell build fix #7

All source soce files which first needs to be generated needs to be listed in
BUILT_SOURCES.
This patch adds to that automake variable parse.c and parse.h because they are
listed below in libcgroup_la_SOURCES.

Documentetion of the BUILT_SOURCES is on:
https://www.gnu.org/software/automake/manual/html_node/Sources.html

Reported-by: Tomasz Kłoczko <kloczek@giithub.com>
Signed-off-by: Tomasz Kłoczko <kloczek@giithub.com>

api.c: Declare cgroup_process_v2_mnt() as STATIC

This commit changes cgroup_process_v2_mnt() from static to STATIC
to allow it to be unit tested.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: Declare cgroup_process_v1_mnt() as STATIC

This commit changes cgroup_process_v1_mnt() from static to STATIC
to allow it to be unit tested.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: cgget: cgset: Add cgroup v2 support

This commit adds cgroup v2 support to cgroup_init(). With these
changes, cgget and cgset now work on a cgroup v1 mount, a cgroup
v2 mount, or a cgroup v2 unified mount hierarchy.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Add support for cgroup v2 to cg_test_mounted_fs()

This commit adds a string comparison of "cgroup2" to
cg_test_mounted_fs().  This allows cgroup v2 mounts to pass the
filesystem check and thus cgroups can be created.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: Only stat the cgroup tasks file if it's a v1 mount

In cgroup_get_cgroup(), stat should only be called on the cgroup's
task file if the cgroup is a v1 mount.  v2 mounts don't have a
tasks file.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: Add cgroup version enumeration

This commit adds a cgroup version enumeration to the
cg_mount_table.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

api.c: Refactor cgroup v1 logic in cgroup_init()

In preparation for adding support for cgroup v2, this commit
moves the cgroup v1 mount logic out of cgroup_init() and into
its own function.  No functional changes were made.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Acked-by: Dhaval Giani <dhaval.giani@oracle.com>

Merge remote-tracking branch 'upstream/master' into HEAD

README: Update the code coverage badge

Update the code coverage badge with the latest recommended
markdown from coveralls.io

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

bootstrap: Update bootstrap to support libcgroup-tests

This commit updates bootstrap.sh to manage the tests folder now
that it's a git submodule copy of libcgroup-tests.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

git: Add submodule libcgroup-tests

This commit adds the libcgroup-tests git repo as a submodule.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

tests: Remove the tests folder from this git repo

The test code has been moved to a separate git repo,
libcgroup-tests.  But before that repo can be added as a
submodule, the tests directory must first be deleted from
this repo.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

tests: Remove the tests folder from this git repo

The test code has been moved to a separate git repo,
libcgroup-tests.  Before we can add that repo as a submodule,
we must first delete the tests directory.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge pull request #5 from drakenclimber/issues/fix_dist_check

Fix small issues with distribution zip files

Merge pull request #6 from drakenclimber/codereviews/nforro-old-fixes2

Series of old fixes from Nikola Forró

Increase maximal size of controller values

Maximal length of a controller value is determined by CG_VALUE_MAX,
which is equal to 100. That is not sufficient in some cases.

Add new constant CG_CONTROL_VALUE_MAX (to prevent breaking current API)
and set it to 4096, which is usually equal to the amount of bytes that
can be written to a sysctl file directly.

Add warning message about exceeding the limit while parsing
configuration file.

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix level of failed user/group lookup warnings

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Show warning when tasks file can not be opened

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Preserve dirty flag when copying controller values

When setting cgroup parameters with cgset fails, no error is reported.
This is caused by the fact that cgroup_copy_controller_values is not
preserving dirty flags of the values, so it's making all errors
considered non-fatal.

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix potential buffer overflow

It is assumed that arguments read from /proc/<pid>/cmdline don't exceed
buf_pname buffer size, which is FILENAME_MAX - 1 characters, but that's
not always the case.

Add check to prevent buffer overflow and discard the excessive part of
an argument.

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix order of memory subsystem parameters generated by cgsnapshot

Order of parameters usually doesn't matter, but that's not the case with
memory.limit_in_bytes and memory.memsw.limit_in_bytes. When the latter
is first in the list of parameters, the resulting configuration is not
loadable with cgconfigparser.

This happens because when a cgroup is created, both memory.limit_in_bytes
and memory.memsw.limit_in_bytes parameters are initialized to highest
value possible (RESOURCE_MAX). And because memory.memsw.limit_in_bytes
must be always higher or equal to memory.limit_in_bytes, it's impossible
to change its value first.

Make sure that after constructing parameter list of memory subsystem,
the mentioned parameters are in correct order.

Signed-off-by: Nikola Forró <nforro@redhat.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

tests: Fix test failure on Fedora 30

Unit test 004-cgroup_compare_ignore_rule.cpp segfaults on
test CgroupCompareIgnoreRuleTest.CombinedControllers().  This
is due to rule.procname being uninitialized.  This commit
initializes the entire rule structure to zero.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

autotools: gtest: Distribute the googletest *.so files

googletest strongly recommends that their library is
rebuilt on each platform, but this runs counter to a
tagged release of code that utilizes gtest.  This commit
enables `make dist` to bundle the googletest *.so files
into the resultant zip file.

Note that these googletest *.so files are dependent upon
a certain version of glibc (and other libraries) and may
not work on distros with very old versions of those
tools.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Release v0.42

Merge pull request #4 from drakenclimber/issues/fixdistzip

Fix a few issues with the test Makefile.ams

tests: Ship the functional test files

This commit adds the *.py files to the dist zip so that
the functional tests can be run in the dist zip.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Reviewed-by: Dhaval Giani <dhaval.giani@oracle.com>

gtest: Cleanup gtest make targets

Prior to this commit the unit test map file was being
included in the source directory.  It has now been moved
to the tests/gunit directory.  This commit also properly
connects the tests/gunit Makefile to the googletest *.la
so that a recompile of googletest is not required.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Reviewed-by: Dhaval Giani <dhaval.giani@oracle.com>

bootstrap: Actually build gtest

googletest should be entirely compiled into an *.la
prior to ./configure.  The *.la should be added as
an extra distribution item to the dist tarball.  This
will make it so that the distribution vendors do not
need to rebuild googletest.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Reviewed-by: Dhaval Giani <dhaval.giani@oracle.com>

Merge pull request #3 from drakenclimber/issues/releaseprocess

doc: Add release process for github

doc: Add release process for github

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

Merge branch 'issues/fix_warnings2' of git://github.com/drakenclimber/libcgroup

Signed-off-by: Dhaval Giani <dhaval.giani@oracle.com>

wrapper.c: Fix potentially unterminated strings

This commit fixes two adjacent strncpys that could result
in unterminated strings:

CID 1412144 (#2 of 2): Buffer not null terminated
(BUFFER_SIZE_WARNING)13. buffer_size_warning: Calling strncpy with a
maximum size argument of 100 bytes on destination array
cntl_value->value of size 100 bytes might leave the destination string
unterminated.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

wrapper.c: Fix buffer not null terminated Coverity warning

This patch fixes the following Coverity warning:

CID 1412155 (#1 of 1): Buffer not null terminated
(BUFFER_SIZE_WARNING)4. buffer_size_warning: Calling strncpy with a
maximum size argument of 100 bytes on destination array val->value of
size 100 bytes might leave the destination string unterminated.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix double free in cgroup_create_template_group()

Coverity identified a double free situation:

CID 1412117 (#1 of 1): Double free (USE_AFTER_FREE)7. double_free:
Calling free frees pointer template_name which has already been freed.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

config.c: Silence Coverity warning about unterminated string

This commit silences the following Coverity warning:

CID 1412118 (#1 of 1): Buffer not null terminated
(BUFFER_SIZE_WARNING)6. buffer_size_warning: Calling strncpy with a
maximum size argument of 4096 bytes on destination array mount->name of
size 4096 bytes might leave the destination string unterminated.

Note that there are several areas (including this piece of code)
in libcgroup that do not gracefully handle string truncation.  That
will be addressed in future patches.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

cgrulesengd: Fix Coverity warnings about a socket leak

This commit resolves the following Coverity warnings:

CID 1412123 (#1 of 1): Resource leak (RESOURCE_LEAK)12. leaked_handle:
Handle variable sk_nl going out of scope leaks the handle.

CID 1412129 (#1 of 1): Resource leak (RESOURCE_LEAK)13. leaked_handle:
Handle variable sk_unix going out of scope leaks the handle.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

config.c: Fix Coverity warning about uninitialized pointer read

This commit resolves the following warning from Coverity:

CID 1412139 (#1 of 1): Uninitialized pointer read (UNINIT)5.
uninit_use_in_call: Using uninitialized value ctrl_handle when calling
cgroup_get_controller_end.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix string truncation warning

This commit fixes this warning in api.c:

api.c: In function ‘cgroup_delete_cgroup_ext’:
api.c:2285:51: warning: ‘snprintf’ output may be truncated before the
    last format character [-Wformat-truncation=]
    2285 |    snprintf(child_name, sizeof(child_name), "%s/%s",
         |                                                   ^
api.c:2285:4: note: ‘snprintf’ output 2 or more bytes (assuming 4097)
    into a destination of size 4096
    2285 |    snprintf(child_name, sizeof(child_name), "%s/%s",
         |    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    2286 |      cgroup_name,
         |      ~~~~~~~~~~~~
    2287 |      info.full_path + group_len);
         |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix string truncation warning

This commit fixes this warning in api.c:

api.c: In function ‘cgroup_read_value_begin’:
api.c:4114:47: warning: ‘snprintf’ output may be truncated before the
    last format character [-Wformat-truncation=]
    4114 |  snprintf(stat_file, sizeof(stat_file), "%s/%s", stat_path,
         |                                               ^
api.c:4114:2: note: ‘snprintf’ output 2 or more bytes (assuming 4097)
    into a destination of size 4096
    4114 |  snprintf(stat_file, sizeof(stat_file), "%s/%s", stat_path,
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    4115 |   name);
         |   ~~~~~

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix string truncation warning

This commit fixes this warning in api.c:

api.c: In function ‘cgroup_read_stats_begin’:
api.c:4189:47: warning: ‘.stat’ directive output may be truncated
    writing 5 bytes into a region of size between 0 and 4095
    [-Wformat-truncation=]
    4189 |  snprintf(stat_file, sizeof(stat_file), "%s/%s.stat", stat_path,
         |                                               ^~~~~
api.c:4189:2: note: ‘snprintf’ output 7 or more bytes (assuming 4102)
    into a destination of size 4096
    4189 |  snprintf(stat_file, sizeof(stat_file), "%s/%s.stat", stat_path,
         |  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    4190 |    controller);
         |    ~~~~~~~~~~~

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix warning caused by misplaced const keyword

This commit fixes this warning in api.c:

api.c:267:34: warning: duplicate ‘const’ declaration specifier
    [-Wduplicate-decl-specifier]
    267 |   int owner_is_umask, const char const **ignore_list)

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix warning caused by misplaced const keyword

This commit fixes this warning in api.c:

api.c:117:19: warning: duplicate ‘const’ declaration specifier
    [-Wduplicate-decl-specifier]
    117 | static const char const *cgroup_ignored_tasks_files[] = {
    "tasks", NULL };

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

api.c: Fix warning caused by misplaced const keyword

This commit fixes this warning in api.c:

api.c:84:12: warning: duplicate ‘const’ declaration specifier
    [-Wduplicate-decl-specifier]
    84 | const char const *cgroup_strerror_codes[] = {

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

cgrulesengd: Fix improper indentation warning

This commit fixes the following warning:

cgrulesengd.c: In function ‘cgre_parse_syslog_facility’:
cgrulesengd.c:1015:5: warning: this ‘if’ clause does not guard...
    [-Wmisleading-indentation]
    1015 |     if (strlen(arg) > 1)
         |     ^~
cgrulesengd.c:1018:2: note: ...this statement, but the latter is
    misleadingly indented as if it were guarded by the ‘if’
    1018 |  switch (arg[0]) {
         |  ^~~~~~

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

parse.y: Fix type declaration for group_name

The group_name grammar rule was incorrectly defined as
returning a type<val> which is an int.  It actually returns
a type<name> which is a char *.

Having group_name declared as val (int) and assigning a char * value
to it can lead to crashes on platforms where pointer size exceeds
size of int.

On newer versions of Bison, this bug led to the following
warnings during compilation:

parse.y: In function ‘yyparse’:
parse.y:106:56: warning: passing argument 1 of
    ‘cgroup_config_insert_cgroup’ makes pointer from integer
    without a cast [-Wint-conversion]
    106 |    $$ = cgroup_config_insert_cgroup($2);
        |                                                        ^
        |                                                        |
        |                                                        int

In file included from parse.y:21:
./libcgroup-internal.h:231:39: note: expected ‘char *’ but argument is
    of type ‘int’
    231 | int cgroup_config_insert_cgroup(char *cg_name);
        |                                 ~~~~~~^~~~~~~

parse.y:125:15: warning: assignment to ‘int’ from ‘char *’ makes integer
    from pointer without a cast [-Wint-conversion]
    125 |   $$ = $1;
        |               ^

parse.y:129:15: warning: assignment to ‘int’ from ‘char *’ makes integer
    from pointer without a cast [-Wint-conversion]
    129 |   $$ = $1;
        |               ^

parse.y: In function ‘yyparse’:
parse.y:106:56: warning: passing argument 1 of
    ‘cgroup_config_insert_cgroup’ makes pointer from integer without a cast
    [-Wint-conversion]
    106 |    $$ = cgroup_config_insert_cgroup($2);
        |                                                        ^
        |                                                        |
        |                                                        int

In file included from parse.y:21:
./libcgroup-internal.h:231:39: note: expected ‘char *’ but argument is
    of type ‘int’
    231 | int cgroup_config_insert_cgroup(char *cg_name);
        |                                 ~~~~~~^~~~~~~
parse.y:125:15: warning: assignment to ‘int’ from ‘char *’ makes integer
    from pointer without a cast [-Wint-conversion]
    125 |   $$ = $1;
        |               ^

parse.y:129:15: warning: assignment to ‘int’ from ‘char *’ makes integer
    from pointer without a cast [-Wint-conversion]
    129 |   $$ = $1;
        |               ^

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>
Signed-off-by: Nikola Forró <nforro@redhat.com>
Acked-by: Michal Koutný <mkoutny@suse.com>

Tag new release 0.42.rc1

Signed-off-by: Dhaval <dhaval@localhost.localdomain>

 tests: Add unit tests for cgroup_compare_wildcard_procname()

This commit adds unit tests for cgroup_compare_wildcard_procname().

The following tests are included in this commit:

WildcardProcnameSimpleMatch - Compares an ignore rule with a
    wildcarded procname with a matching procname.  Expects true.

WildcardProcnameNoMatch - Compares an ignore rule with a wildcarded
    procname with a process that has a non-matching name.  Expects
    false.

ProcnameWildcard_AsteriskNoMatch - Compares a wildcard rule with
    a process name that doesn't match.  Expects false.

ProcnameWildcard_AsteriskMatch - Compares a wildcard rule with
    a process name that does match the rule.  Expects true.

ProcnameWildcard_AsteriskNoMatch2 - Compares a wildcard rule
    with a process name that is shorter than the rule and thus
    does not match.  Expects false.

ProcnameWildcard_AsteriskMatchExactly - Compares a wildcard
    rule with a process name that matches the rule exactly.
    Expects true.

ProcnameWildcard_NoAsteriskMatchExactly - Compares a rule
    with no wildcards with a process name that exactly matches
    the rule.  Expects false as the function exits early due
    to no asterisk in the rule.

The results from googletest are reported below:

[----------] 13 tests from CgroupCompareIgnoreRuleTest
<snip>
[ RUN      ] CgroupCompareIgnoreRuleTest.WildcardProcnameSimpleMatch
[       OK ] CgroupCompareIgnoreRuleTest.WildcardProcnameSimpleMatch (0 ms)
[ RUN      ] CgroupCompareIgnoreRuleTest.WildcardProcnameNoMatch
[       OK ] CgroupCompareIgnoreRuleTest.WildcardProcnameNoMatch (0 ms)
[----------] 13 tests from CgroupCompareIgnoreRuleTest (1 ms total)

[----------] 7 tests from ProcnameWildcardTest
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_NoAsterisk
[       OK ] ProcnameWildcardTest.ProcnameWildcard_NoAsterisk (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_AsteriskNoMatch
[       OK ] ProcnameWildcardTest.ProcnameWildcard_AsteriskNoMatch (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_AsteriskMatch
[       OK ] ProcnameWildcardTest.ProcnameWildcard_AsteriskMatch (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_AsteriskNoMatch2
[       OK ] ProcnameWildcardTest.ProcnameWildcard_AsteriskNoMatch2 (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_AsteriskMatchExactly
[       OK ] ProcnameWildcardTest.ProcnameWildcard_AsteriskMatchExactly (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_NoAsteriskMatchExactly
[       OK ] ProcnameWildcardTest.ProcnameWildcard_NoAsteriskMatchExactly (0 ms)
[ RUN      ] ProcnameWildcardTest.ProcnameWildcard_AsteriskFirstChar
[       OK ] ProcnameWildcardTest.ProcnameWildcard_AsteriskFirstChar (0 ms)
[----------] 7 tests from ProcnameWildcardTest (0 ms total)

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>

cgrulesengd: Add wildcard matching for process names

This commit adds wildcard matching to process name
parsing in cgrulesengd.  Note that wildcard matching
works for standard rules and ignore rules.

For example, given the following rule in cgrules.conf

\#<user> <controllers> <destination>
tom:foo* memory FooCG/

Processes named 'foo', foo1', 'foo2', etc. would be moved
to the FooCG/ cgroup.  Processes named 'bar', 'fo', etc.
would not match this rule.

Signed-off-by: Tom Hromatka <tom.hromatka@oracle.com>