Bug 16672 - [src.rpm] Incomplete package if built as non-root
Summary: [src.rpm] Incomplete package if built as non-root
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: krb5
Version: 6.2
Hardware: i386
OS: Linux
medium
low
Target Milestone: ---
Assignee: Nalin Dahyabhai
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2000-08-21 12:25 UTC by Enrico Scholz
Modified: 2008-05-01 15:37 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2000-08-21 13:49:10 UTC
Embargoed:


Attachments (Terms of Use)

Description Enrico Scholz 2000-08-21 12:25:19 UTC
When building the package as non-root something goes wrong with the `ksu'
program (see 4th and last line):

--------------
making install in clients/ksu...
make[2]: Entering directory
`/usr/src/redhat/BUILD/krb5-1.2.1/src/clients/ksu'
for f in ksu; do \
  /usr/bin/install -c -s -m 4755 -o root $f \
        /var/tmp/krb5-root/usr/kerberos/bin/`echo $f|sed 's,x,x,'`; \
  /usr/bin/install -c -m 644 ./$f.M \
        /var/tmp/krb5-root/usr/kerberos/man/man1/`echo $f|sed 's,x,x,'`.1;
\
done
/usr/bin/install: /var/tmp/krb5-root/usr/kerberos/bin/ksu: Operation not
permitted
--------------

Comment 1 Nalin Dahyabhai 2000-08-21 13:48:36 UTC
Our build system defaults to building packages as a non-root user specifically
to catch these cases.  Does the ksu binary still end up in the right place?

Comment 2 Nalin Dahyabhai 2000-08-21 14:10:23 UTC
I've just rebuilt the package as myself, and the files get included.  The error
message is harmless.

Comment 3 Enrico Scholz 2000-08-21 14:18:56 UTC
But:

# ll /usr/kerberos/bin/ksu 
-rwxr-xr-x    1 root     root        50364 Aug 21 14:22 /usr/kerberos/bin/ksu

The official install-method tries to install it SUID; but the packaged file
isn't it.

Comment 4 Nalin Dahyabhai 2000-08-21 14:24:01 UTC
The MIT Kerberos team isn't sure that the previously-fixed vulnerabilities in
ksu were all of the potential problems, so we took the setuid-bit off pending a
complete audit of the sources.


Note You need to log in before you can comment on or make changes to this bug.