[fix] sessiond: do not follow symlinks while setting SMACK attributes

of the copied files

Change-Id: Ia775dc9af277a333dbd5e75ee8e4ba2eebe88098

diff --git a/packaging/sessiond.spec b/packaging/sessiond.spec
index d41d8bf..54f8e63 100644 (file)
--- a/packaging/sessiond.spec
+++ b/packaging/sessiond.spec
@@ -1,6 +1,6 @@
 Name:       sessiond
 Summary:    Service to manage subsessions
-Version:    0.7.0
+Version:    0.7.1
 Release:    1
 Group:      System/Management
 License:    MIT

diff --git a/sessiond/src/fs_helpers.cpp b/sessiond/src/fs_helpers.cpp
index 7a43ea4..35946c4 100644 (file)
--- a/sessiond/src/fs_helpers.cpp
+++ b/sessiond/src/fs_helpers.cpp
@@ -81,7 +81,7 @@ void fs_helpers::copy_ownership(std::string_view src_path, std::string_view dest
 std::string fs_helpers::get_smack_label(std::string_view src_path, smack_label_type type)
 {
        char *label_raw = nullptr;
-       int ret = smack_getlabel(src_path.data(), &label_raw, type);
+       int ret = smack_lgetlabel(src_path.data(), &label_raw, type);
 
        std::unique_ptr<char, decltype([] (char *p) {
                free(p);
@@ -114,10 +114,10 @@ void fs_helpers::copy_smack_attributes(std::string_view src_path, std::string_vi
                        // N.B. Setting TRANSMUTE attribute needs special attention:
                        // the only correct values are: NULL, "", "0" or "1".
                        if (label == "TRUE")
-                               ret = smack_setlabel(dest_path.data(), "1", type);
+                               ret = smack_lsetlabel(dest_path.data(), "1", type);
                }
                else
-                       ret = smack_setlabel(dest_path.data(), label.c_str(), type);
+                       ret = smack_lsetlabel(dest_path.data(), label.c_str(), type);
 
                if (ret)
                        throw std::runtime_error(