Mailing List Archive: [Bug 503] dsearch: use lstat() instead of stat()

[Bug 503] dsearch: use lstat() instead of stat()

May 20, 2007, 8:30 AM

Post #1 of 3 (830 views)

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

http://www.exim.org/bugzilla/show_bug.cgi?id=503

------- Comment #1 from holmgren@lysator.liu.se 2007-05-20 16:30 -------
Sounds reasonable, and pretty unlikely that it will break existing setups. Any
other opinions?

--
Configure bugmail: http://www.exim.org/bugzilla/userprefs.cgi?tab=email

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 503] dsearch: use lstat() instead of stat() [ In reply to ]

exim at lists

May 31, 2007, 12:42 AM

Post #2 of 3 (826 views)

Permalink

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

http://www.exim.org/bugzilla/show_bug.cgi?id=503

exim@lists.colondot.net changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |exim@lists.colondot.net

------- Comment #2 from exim@lists.colondot.net 2007-05-31 08:42 -------
Given that when I originally wrote the dirsearch thing for exim3 I did it as a
readdir, I think that doing an lstat is entirely reasonable.

Bear in mind that in dsearch, you're checking for the name existing in that
directory in general, so dangling symlinks don't seem like a problem to me.

I'd vote for changing it.

Cheers

MBM

--
Configure bugmail: http://www.exim.org/bugzilla/userprefs.cgi?tab=email

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##

[Bug 503] dsearch: use lstat() instead of stat() [ In reply to ]

holmgren at lysator

May 31, 2007, 5:47 AM

Post #3 of 3 (800 views)

Permalink

------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

http://www.exim.org/bugzilla/show_bug.cgi?id=503

holmgren@lysator.liu.se changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED

------- Comment #3 from holmgren@lysator.liu.se 2007-05-31 13:47 -------
It doesn't contradict the spec either. Perhaps the spec should be clarified,
but I don't think it's necessary.

Committed to CVS.

--
Configure bugmail: http://www.exim.org/bugzilla/userprefs.cgi?tab=email

--
## List details at http://www.exim.org/mailman/listinfo/exim-dev Exim details at http://www.exim.org/ ##