[Bug 225288] Merge Review: at

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: at


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225288





------- Additional Comments From tibbs@xxxxxxxxxxx  2007-02-20 10:50 EST -------
Let me offer some comments on the rpmlint output.

W: at strange-permission atd.init 0775
W: at strange-permission test.pl 0755
  Generally these aren't worth bothering with, but having a file group writable
in your checkout could be problematic.  I don't see that, but I think my umask
doesn't allow it.  Someone should try to understand where this is coming from.

E: at non-readable /etc/pam.d/atd 0640
   I think this is OK, albeit different from what most other packages do.  (They
use 0644).

E: at non-standard-dir-perm /var/spool/at/spool 0700
E: at non-standard-dir-perm /var/spool/at 0700
E: at non-readable /etc/at.deny 0600
E: at non-readable /var/spool/at/.SEQ 0600
   These are necessitated by security.

W: at hidden-file-or-dir /var/spool/at/.SEQ
   That's just the file that at uses; it's OK for it to be hidden.

E: at setuid-binary /usr/bin/at root 04755
E: at non-standard-executable-perm /usr/bin/at 04755
   These are necessary.

W: at dangerous-command-in-%post chown
   I'm not really sure why these are here as opposed to just being part of
%files.  Perhaps rpm would keep creating .SEQ.rpmnew files endlessly otherwise?
 If so then I think it's OK.

W: at service-default-enabled /etc/rc.d/init.d/atd
   It's allowable for a service to be on by default, especially in the case of a
daemon that everyone expects to be there.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]