Hi! On 12/08/2014 09:02 PM, Cyril Hrubis wrote: > Hi! >>> Will simple access() suffice here or do we have to actually open the >>> file? >> We have to open it, it was changed from access to open in this commit: >> >> commit 0a91955ab91bf518ff56a44bb2969b0cb549a979 >> Date: Wed May 7 17:03:33 2014 +0400 >> >> device-drivers/acpi/ltp_acpi_cpufreq: fix checking for write permission >> >> If access(some_file, W_OK) is executed by root it will return >> success even >> if file has read-only permission. Use 'open' to test sysfs file for >> write >> permission. >> >> Doc updated as well: >> >> commit fb3da0a1f3a4f1b836e50033cb8acf7c6d0c5a05 >> Date: Tue May 13 15:34:32 2014 +0400 >> >> docs: add a paragraph about access() to Common Problems section > Ah, right, deja vu. >
Both patches pushed. Thanks, Alexey ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=164703151&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list
