Use smack_label_length() internally instead strnlen()

Use smack_label_length() to check correctness labels instead strnlen().
(cherry picked from commit 6d3fd3049e08377f75b554cedcd200286fbf359c)

Fix value returned by smack_label_length()

In some cases smack_label_length have returned 0 on incorrect label.
Now smack_label_length() always returns -1 in case of incorrect label.
(cherry picked from commit f7e4232a319b269f6214af660173eebaa605ad9e)

Remove errno assignments

If we want to report something libsmack specific, then we should
have our own error codes. By using errno values to report libsmack
specific error condition we almost zero their value. By removing
these assignment you can resolved from a changed errno value that
a system call failed.

This patch does not break API contract as use of errno values is
undefined in the API documentation.
(cherry picked from commit 1edba541de63b088a60d89e4a0433bf9149c4b13)

Removed smack_label_length() API

In order to use this smack_label_length() in 1.0 branch, this patch
takes it away from the API. Adds temporarily unused attribute to the
function siganture.

Add smack_label_length() function

Kernel does not validate the Smack label - instead the label will be
cut on the first incorrect character (after parsing at least one
correct character).

This function gives the user possibility to verify the correctness
of the Smack label before use and calculate labels length.

Additionally, all string length calculations are replaces with this
function to make implementation safer.

[jarkko.sakkinen@linux.intel.com:
 did some modifications:
 - smack_is_label_valid() -> smack_label_length()
 - libsmack.c:
   * return length
   * loop invariant had off-by-one error
   * cosmetic: "++i" not "i++"
 - libsmack.h:
   * updated documentation
   * cosmetic: formatting errors in  @param and @return]
(cherry picked from commit 8b083a8c67219c5d1dbfbf2ad1082c1954f9c9fa)

Cleaning error logs for rules applying utilities

Removed perror logs from main utilities files. Moved error logs
inside /utils/common.c. Errors are logged from I/O operations and
applying rules/cipso from files.
(cherry picked from commit bcaf9018f41809cb93aaccf9e9fb49b59750dc7e)

Add parameter name back to the chsmack error message

This patch add parameter name back to the error message. Instead
of relaying on hazardous longindex parameter of getopt_long(), a
look-up table is constructed to map short option to the corresponding
struct option entry.

Additionally, the basename of the application is added to the error
message. Also usage message is converted to use basename instead of
the full path name.
(cherry picked from commit d98a04ff7ccbcfcf36d9d4d43de68448b70da9fa)

utils/chsmack: fix hazardous option parsing

The variable option_index was only set to a proper value if the
given option is a long option. There was also exit() missing on
error condition if SMACK label was invalid.

This patch removes option_index, parameter name from corresponding
error message and adds exit() call when the error condition
realizes.

[jarkko.sakkinen@linux.intel.com: rewrote patch description]
(cherry picked from commit 18ebc2fe71da94599e45029d7b26144c6bbe7cb4)

Fix: accept 255 character labels

Example:

 # /home/jsakkine/devel/smack/utils/chsmack \
   -a$(printf '12345%.0s' {1..51}) foo
 foo: Invalid argument
 # /home/jsakkine/devel/smack/utils/chsmack \
   -a$(printf '12345%.0s' {1..51} | head -c -1) foo
 #

This patch fixes this issue.
(cherry picked from commit 1794fed08418b0e254e1e3a7325a8a67f8186bbc)

Change apply_cipso_cb() to use proper apply_cipso_file()

apply_cipso_cb() used improper apply_rules_file(). Changed this to
call apply_cipso_file() instead.
(cherry picked from commit d45ffe9082243377cdba9a2d224bab0e110501e0)

Fix: allow CIPSO labels with zero categories

There was false restriction in smack_cipso_add_from_file() that
disallowed CIPSO labels without categories.

For example, this example given in the SMACK kernel documentation
should be perfectly legal:

TopSecret 7
(cherry picked from commit 4e4ea9142727ca7f14bf1d64cd81949b28bb1d0b)

Documented order in which access rules are applied to kernel.
(cherry picked from commit f82cadf3b7a9d84ae0bf2aa06fb8ecff9b4332dc)

Fixes for doxygen errors coming from 3810596.

Doxygen errors that are regression from 3810596. Went in because
I forgot to install doxygen into my test environment.
(cherry picked from commit 8ef1846c05d32a92b23538ee491fd31350c2a045)

libsmack: clean up smack_accesses_add_modify()

This patch makes smack_accesses_add_modify() easier to follow by
changing variable names bit more self-documentative and less easier
to mix up with adding and deleting rules. This patch also fixes some
very minor coding style errors.

Also documentation of this function is cleaned up.
(cherry picked from commit 22c98dd03e352057b23f8d8d2d9ad252175c685b)

Cleaned up init code.

There was lot of clutter and misleading comments in the code
and some minor coding convention issues. This patch cleans up
init.c for the most part.
(cherry picked from commit c59864f9d4a67dcf76ba01d9c4db2b12fada6339)

Update configure.ac version and mainter information

Update version to 1.0.2 and maintaer email from @intel.com
to @linux.intel.com.

Set revision for v1.0.2 release

This was forgotten to set for v1.0.1. My apologies.
Setting this early on so will not be forgotten when
v1.0.2 is tagged.

Major documentation cleanup.

smack.h was starting to rotten so I did a major cleanup for it.
Also fixed couple of minor doxygen issues by removing deprecated
variables.

Removed redundant is_smackfs_mounted().

This functionality is fully handled in the library initialization.

Revert "Avoid memory allocation while opening smackfs files."

This reverts commit 0426039bffdb82b0f7974639c1c1e361929b1cd5.

Avoid memory allocation while opening smackfs files.

Using openat() on pre-opened smackfs directory eliminates need to construct
absolute path to a smackfs file before opening it.

Prevent potentially unterminated buffers while adding rule to the list

Functions smack_accesses_add() and smack_accesses_add_modify() don't check
length of arguments subject and object. These arguments are used as source
for strncpy(), which can cause labels to be truncated.

But the length argument for strncpy() is too large. This might cause
rule->subject or rule->object to be not terminated by null character.
It can happen when these functions are called from outside libsmack.
It can also happen while parsing files in smack_accesses_add_from_file(),
because that function doesn't validate subject and object too.

This commit fixes the problem by checking arguments in smack_accesses_add()
and smack_accesses_add_modify(). After checking strcpy() is safe for
copying them.

Added AUTHORS file.

Cleaned up command line arguments handling in chsmack.

Argument handling code looks terrible at the moment in chsmack.
Migrated to getopt_long().

Removed doc/smackd.8 from debian/smack-utils.manpages

LIBSMACK -> LIBSMAC_1.0 in libsmack.sym

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Added smackctl back.

Added smackctl back because Debian needs it.

Documentation fixes

* Added doc/smack_new_label_from_path.3.
* Fixed doxygen warnings from smack.h.

Removed smackd and smackctl.

For shell usage smackload is enough so we don't want to be these as
part of the 1.0 release. systemd has built-in support for loading
SMACK rules.

Merge pull request #12 from rafal-krypa/samsung

Use xattr.h from libc, not from libattr.

smack_new_label_from_path fix: removed const specifier

Label parameter had const specifier by mistake.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Added missing smack_new_label_from_path symbol to libsmack.sym

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Return length of the label on success in smack_new_label_from*

Return length of the label on success and negative value on
failure.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

fix: smack_new_from_path() follow parameter

Mistake in smack_new_from_path(). When follow is true getxattr()
should be called and lgetxattr() otherwise.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Use xattr.h from libc, not from libattr.

This fixes include in libsmack.c introduced in commit f409c17.

Added follow parameter to smack_new_label_from_path()

Added follow parameter to smack_new_label_from_path() so that it
is useful for coreutils commands.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Added function smack_label_from_path()

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Finalized cipso API.

Aligned cipso API with accesses API so that they have same style.
Added documenation comments for cipso API.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Merge pull request #11 from rafal-krypa/samsung

Samsung

libsmack: add support for modification rules.

A new API function smack_accesses_add_modify() is provided for adding
modification rule to a set. Rule specifies permissions to grant and
permissions to revoke. If a rule for given subject and object already
existed, unmentioned permissions will be left unchanged.
Also smack_access_save() and smack_access_add_from_file() are modified
to support reading and writing such rules from/to a file.

libsmack: parse whole access type string, not only first 5 bytes.

Previous version of this function parsed only first ACC_LEN (5)
characters of access_type. Now the whole string will be read.

This will prevent silent ignoring of access type characters in cases
like "-rwxat" or "rrwxat".

Merge pull request #10 from banada/master

disable services for systemd versions > 198

disable services for new systemd versions

do not install smack.mount or smack.service if systemd is version
198 or higher. This functionality is now built into systemd.

Merge pull request #9 from PassionZhao/master

Fix the bug "smackcipso can't set CIPSO"

smackcipso can't set CIPSO correctly, see the bug report:
        https://tizendev.org/bugs/browse/TDIS-3891

The reason is that upstream libsmack only use "smackfs/cipso2" long-label interface,
but the code's logical is still for orginal fixed length label.

Now update smack_cipso_apply() to support flexible label (<=256 including tailing '\0')

Signed-off-by: Passion,Zhao <passion.zhao@intel.com>

Merge pull request #8 from rafal-krypa/samsung

libsmack: check label length in smack_revoke_subject().

libsmack: check label length in smack_revoke_subject().

Merge pull request #7 from phmccarty/master

Run AM_PROG_AR to fix build with newer automake

Run AM_PROG_AR to fix build with newer automake

libsmack: add function for setting current process label.

libsmack: add API for revoking all rules for a subject label.

Fix three compilation warnings.

Fix paths to upstream repositories.

Move definition of max label length to a single place and export it in
libsmack header file.

Install missing manual page links for library functions.

Removed dso.h

We don't need this because we're using version script. Fixed init.c
accordingly.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@iki.fi>

Declare smack_mnt as non-static in init.c.

Use it directly in libsmack.c. It will be declared as local symbol
in DSO because we declare visible symbols explicitly in libsmack.sym.

Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@iki.fi>

Add support for smackfs directory: /sys/fs/smackfs/

Signed-off-by: Passion,Zhao <passion.zhao@intel.com>
Suggested-by: Jarkko Sakkinen <jarkko.sakkinen@intel.com>

Merge pull request #2 from PassionZhao/master

 Move cipso_free,cipso_new,cipso_apply from utils/common.c to libsmack/libsmack.c

Move cipso_free,cipso_new,cipso_apply from utils/common.c to libsmack/libsmack.c
In libsmack/libsmack.c, move smack_cipso-* right after smack-access-*

Merge pull request #1 from sofar/fixes

smack.service: provide [Install] section in systemd unit file.

smack.mount: "WantedBy" is illegal in [Unit] context.

There is no need to define a dependency or preferred mounting
point as smack.service already requires smack.mount to be
started, and smack.service forces itself to be part of
basic.target.

WantedBy additionally is illegal in [Unit] and so does
nothing but create a warning.

Since mounting smackfs without enabling smack.service doesn't
really do much, we opt out of putting Also=smack.mount in
smack.service.

smack.service: provide [Install] section in systemd unit file.

With this addition, one can `systemctl --enable smack.service`
to enable smack.service and smack.mount at boot time.

libsmack: fallback to short labels

Use /smack/load and /smack/access as a fallback if kernel does not
have long label support.

Update to latest Doxygen standard

Must umount /smack before trying to remove it

Optimize init script smack.rc

Add postinst + postrm scripts for Debian to create /smack directory

Update copyright file for Debian

Add libsmack1.symbols file for Debian

Full update of Debian control file

Rename Debian package name libsmack to libsmack1

Create manpage links to existing manpages

Fix warnings

Utils: add debian init script and required directories

Fix some typos in manpages

Signed-off-by: Joachim Wiedorn <ad_debian@joonet.de>

Add man pages

Utils: Add long label support

libsmack: Add long label support

libsmack: Ignore blank lines in the rules files

Init: Dynamically check systemd installation path

Add initial debian package support

Remove unnecessary description from perror as there only exists one call to each method

utils: improve smackcipso so that it loads also from stdin.

Original smackutil smackcipso allowed to read cipso from
stdin. This adds the same feature here.

Conflicts:

utils/smackcipso.c

Add init scripts

Merge pull request #10 from brianmcgillion/master

Include guard fix

Improve the include guard to avoid conflicts with applications

fixed libsmack.sym

libsmack: revert smack_new_label_from_self()

Reverted smack_new_label_from_self() back. Removed
smack_set_self_label as it is not needed.

utils: stdin support for smackload

utils: cleaned up common code for applying rules.

Added smack_get_self_label().

Added smack_set_self_label().

Fix: common.h was not part of the dist.

utils/Makefile.am: fixed installation path.

Merge pull request #5 from brianmcgillion/master

extern keyword missing

Merge branch 'master' of https://github.com/jarkkos/smack

libsmack: Fix missing extern qualifier

Merge pull request #4 from brianmcgillion/ac028f67ba540b74d2f7d9b169ceaff548ba5f43

Add chsmack

Merge branch 'master' of https://github.com/jarkkos/smack