Landscape Client

  1. Bug #362355
  2. Comment #16

Comment 16 for bug 362355

Revision history for this message
Free Ekanayaka (free.ekanayaka) wrote : Re: [Bug 362355] Re: Run smart update in the package-reporter instead of having a cronjob

Thanks for the review, Chris!

[1]

Nice, thanks for hint.

[2]

Fixed.

[3]

  > setreuid(pwd->pw_uid, pwd->pw_uid);
  > setregid(pwd->pw_gid, pwd->pw_gid);
  > Is it really necessary? I guess it's a protection in case the program

AFAIK, it is necessary to actually set the effective uid to 0. OTOH I
don't think setresuid is necessary, because the purpose of the setuid
permission flag is indeed to set the saved uid of the process to the
owner of the file once the OS executes it.