Menu

#49 Add invalid path /etc not pointing to /etc/opt/csw

None
new
wahwah
None
checkpkg
major
enhancement
2011-10-25
2011-04-28
No

I just noticed a build which references a configuration file in /etc instead of /etc/opt/csw:

current9s% cd work/solaris9-sparc/build-isa-sparcv8/bar-1.11.1/
current9s% grep clpbarrc *
args.c: if (access("/etc/clpbarrc", R_OK) == 0) {
args.c:         sprintf(filename, "/etc/clpbarrc");
bar.1:.B /etc/clpbarrc
bar.1:.BR /etc/clpbarrc .
bar.1:.BR /etc/clpbarrc .
bar.1.in:.B /etc/clpbarrc
bar.1.in:.BR /etc/clpbarrc .
bar.1.in:.BR /etc/clpbarrc .

It would be good if checkpkg would catch that.

Discussion

  • wahwah

    wahwah - 2011-04-30

    How would you generalize the check based on the above example?

     
  • Dagobert Michelsen

    Maybe a match to /etc but not /etc/opt/csw?

     

Log in to post a comment.

MongoDB Logo MongoDB