diff options
author | Stephan Gerhold <stephan@gerhold.net> | 2022-07-08 14:50:54 +0200 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2022-07-08 17:31:58 +0200 |
commit | cfd3da47fa0f094d541ff596c1a0c24e6dc9d3e2 (patch) | |
tree | d267a1f073838f58bc77dfd8f201895fca40fb6e | |
parent | 351a237b2df497e78ac2d4666040e02118e924e8 (diff) | |
download | alsa-lib-cfd3da47fa0f094d541ff596c1a0c24e6dc9d3e2.tar.gz |
ucm: fix st_mode check for symbolic links
The file type in the st_mode field is not encoded as unique bits but as
an enumerator. Checking if some bits of S_IFLNK are set does not work
correctly because it happens to evaluate to true for regular files as
well.
The POSIX man page suggests using the following approach to check
the file type:
if ((sb.st_mode & S_IFMT) == S_IFLNK)
Alternatively, there is a S_ISLNK() macro to check this more easily.
Make use of the latter so that readlink() is only called on actual
symbolic links and not regular files. This makes audio work again
with slightly older alsa-ucm-conf versions or alternative top-level
ucm.conf configurations that do not make use of symlinks.
Fixes: d6adde0e ("ucm: top-level path - set directory from symlink")
Fixes: https://github.com/alsa-project/alsa-lib/pull/249
Signed-off-by: Stephan Gerhold <stephan@gerhold.net>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
-rw-r--r-- | src/ucm/parser.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/ucm/parser.c b/src/ucm/parser.c index 351b7c40..976aa8be 100644 --- a/src/ucm/parser.c +++ b/src/ucm/parser.c @@ -2630,7 +2630,7 @@ static int parse_toplevel_path(snd_use_case_mgr_t *uc_mgr, ucm_filename(fn, sizeof(fn), version, dir, file); if (access(fn, R_OK) == 0 && lstat64(fn, &st) == 0) { - if (st.st_mode & S_IFLNK) { + if (S_ISLNK(st.st_mode)) { ssize_t r; char *link, *dir2, *p; |