On Mon, Feb 7, 2011 at 9:11 AM, Florent Thoumie <f...@freebsd.org> wrote: > On Mon, Feb 7, 2011 at 7:52 AM, Doug Barton <do...@freebsd.org> wrote: >> flz, >> >> In r206043 you converted fexists() in src/lib/libpkg/file.c to use open() >> instead of lstat(). Unfortunately this has the side effect of breaking >> 'pkg_info -g' for unprivileged users with files that have no +r bits. For >> example: >> >> pkg_info -g sudo-1.7.4.6 >> Information for sudo-1.7.4.6: >> >> Mismatched Checksums: >> pkg_info: /usr/local/bin/sudo doesn't exist >> pkg_info: /usr/local/bin/sudoedit doesn't exist >> pkg_info: /usr/local/bin/sudoreplay doesn't exist >> pkg_info: /usr/local/sbin/visudo doesn't exist > > How can you compute the checksum if you don't have read access? > I understand that the error message should rather be "Cannot open %s". > >> Reverting your change produces the expected behavior. So my questions are, >> why was the change made, what are its benefits, and how can we fix this >> problem? :) > > I'm sure there was a good reason at the time. I'll have a better look later.
I guess that it is related to the problem where the symbolic link exists but the file pointed by the symlink does not. Why open() and not stat(), I don't remember. > > -- > Florent Thoumie > f...@freebsd.org > FreeBSD Committer > _______________________________________________ > freebsd-ports@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-ports > To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org" > _______________________________________________ freebsd-ports@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-ports To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"
