pkg_info -g not working for unprivileged user + unreadable files

Julien Laffaye jlaffaye at freebsd.org
Mon Feb 7 11:15:12 UTC 2011


On Mon, Feb 7, 2011 at 9:11 AM, Florent Thoumie <flz at freebsd.org> wrote:
> On Mon, Feb 7, 2011 at 7:52 AM, Doug Barton <dougb at 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
> flz at FreeBSD.org
> FreeBSD Committer
> _______________________________________________
> freebsd-ports at freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-ports
> To unsubscribe, send any mail to "freebsd-ports-unsubscribe at freebsd.org"
>


More information about the freebsd-ports mailing list