[Bug 249522] Review Request: sepostgresql - Security-Enhanced PostgreSQL

[bugzilla at redhat.com]

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

Summary: Review Request:  sepostgresql - Security-Enhanced PostgreSQL


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





------- Additional Comments From kaigai at kaigai.gr.jp  2007-08-02 00:08 EST -------
(In reply to comment #6)
> did you successfully build this in mock?  there seem to be missing 
buildrequires
> when i attempt it.

Your patch intended to add "Requires(pre): shadow-utils", but it is not 
necessary because we can assume some fundamental packages are installed.
See, http://fedoraproject.org/wiki/Packaging/FullExceptionList
The shadow-utils is also contained the list.

> current rpmlint output:
> E: sepostgresql non-standard-uid /var/lib/sepgsql sepgsql
> E: sepostgresql non-standard-gid /var/lib/sepgsql sepgsql
> E: sepostgresql non-standard-dir-perm /var/lib/sepgsql 0700
> E: sepostgresql non-standard-uid /var/lib/sepgsql/data sepgsql
> E: sepostgresql non-standard-gid /var/lib/sepgsql/data sepgsql
> E: sepostgresql non-standard-dir-perm /var/lib/sepgsql/data 0700
> E: sepostgresql non-standard-uid /var/lib/sepgsql/backups sepgsql
> E: sepostgresql non-standard-gid /var/lib/sepgsql/backups sepgsql
> E: sepostgresql non-standard-dir-perm /var/lib/sepgsql/backups 0700

The rpmlint generate same errors, except for paths, for the postgresql-server 
package.
I don't think it's a problem.

> W: sepostgresql incoherent-version-in-changelog 8.2.4-0.407 8.2.4-
0.407.beta.fc8

> W: sepostgresql dangerous-command-in-%postun userdel
As I mentioned above, it's commented out.

> E: sepostgresql subsys-not-used /etc/rc.d/init.d/sepostgresql
/etc/rc.d/init.d/sepostgresql uses pg_ctl to start/stop SE-PostgreSQL server 
process, and it creates lock file under the /var/lib/sepgsql.
Should it be fixed?


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.