Post by Klaus SchmidingerPost by Lucian Muresan[...]
Post by Klaus SchmidingerPost by Lucian MuresanWell, with this patch, symbolic links are not displayed at all on my
VDR machine, whereas with sundararaj reel's fix they are displayed
correctly.
So what you're saying it that this...
----------------------------------------------------------------------------------------------
--- a/recording.c
+++ b/recording.c
@@ -1120,9 +1120,13 @@ void cRecordings::ScanVideoDir(const char
*DirName, bool Foreground, int LinkLev
continue;
}
Link = 1;
+#if 0
+ // do not resolve the symbolic links in paths to real path
+ // thereby keeping all the recordings under one directory
buffer = ReadLink(buffer);
if (!*buffer)
continue;
+#endif
if (stat(buffer, &st) != 0)
continue;
}
----------------------------------------------------------------------------------------------
...works, while this...
----------------------------------------------------------------------------------------------
--- recording.c 2012/01/25 09:32:39 2.45
+++ recording.c 2012/01/26 10:02:29
@@ -1120,11 +1120,6 @@
continue;
}
Link = 1;
- buffer = ReadLink(buffer);
- if (!*buffer)
- continue;
- if (stat(buffer, &st) != 0)
- continue;
}
if (S_ISDIR(st.st_mode)) {
if (endswith(buffer, deleted ? DELEXT : RECEXT)) {
----------------------------------------------------------------------------------------------
...doesn't?
I find that hard to believe, because the only difference here is that
the second version removes the stat() call, which is superfluous if
'buffer' is no longer modified.
Haven't really looked at the code, until now, and I also do not
exactly know what the call to stat does and also didn't try to
understand the whole picture now, but your patch does not simply
remove just the call to stat, but also a *continue* statement from the
*while* loop, this could have strong
implications, so just please consider analyzing the issue with respect to that.
The original code was
----------------------------------------------------------------------------------------------
if (stat(buffer, &st) == 0) {
int Link = 0;
if (S_ISLNK(st.st_mode)) {
if (LinkLevel > MAX_LINK_LEVEL) {
isyslog("max link level exceeded - not scanning %s", *buffer);
continue;
}
Link = 1;
buffer = ReadLink(buffer);
if (!*buffer)
continue;
if (stat(buffer, &st) != 0)
continue;
}
...
}
----------------------------------------------------------------------------------------------
After Sundararaj's patch it looked like this (just leaving out the lines
----------------------------------------------------------------------------------------------
if (stat(buffer, &st) == 0) {
int Link = 0;
if (S_ISLNK(st.st_mode)) {
if (LinkLevel > MAX_LINK_LEVEL) {
isyslog("max link level exceeded - not scanning %s", *buffer);
continue;
}
if (stat(buffer, &st) != 0)
continue;
}
...
}
----------------------------------------------------------------------------------------------
Reducing this to the stat() calls results in
----------------------------------------------------------------------------------------------
if (stat(buffer, &st) == 0) {
...
if (stat(buffer, &st) != 0)
continue;
}
...
}
----------------------------------------------------------------------------------------------
As you can see, 'buffer' is no longer modified between the two calls, so they
will both return the same value. The code sequence is only entered if the first
stat() call returns 0, so the second call will also return 0, and thus the
'continue' statement will never be executed.