Opened 16 years ago

Closed 11 years ago

#204 closed defect (fixed)

chown/chgrp functions are only stubs, proposed fix

Reported by: YDARIO Owned by: Yuri Dario
Priority: normal Milestone: libc-0.6.6
Component: libc-backend Version: 0.6.2
Severity: normal Keywords: chgrp chown
Cc:

Description

The two functions chown/chgrp are only stubs for validating file path and nothing more.

The attached code implements setting/getting the two fields; it misses code validation for current user, to check if he has enough rights to operate.

Attachments (3)

chown-grp.txt​ (15.2 KB ) - added by YDARIO 16 years ago.
Patch file for chown/chgrp
patch-204b.txt​ (16.8 KB ) - added by Yuri Dario 11 years ago.
patch-204t.txt​ (16.5 KB ) - added by Yuri Dario 11 years ago.

Download all attachments as: .zip

Change History (12)

by YDARIO, 16 years ago

Attachment: chown-grp.txt​ added

Patch file for chown/chgrp

comment:1 by YDARIO, 16 years ago

I forgot: I implemented the missing functions into existing files to not modify the build environment.

comment:2 by bird, 14 years ago

Status: new → assigned

comment:3 by bird, 13 years ago

Milestone: libc-0.6.5 → libc-0.7

comment:4 by Yuri Dario, 11 years ago

Milestone: libc-0.7 → libc-0.6.6

by Yuri Dario, 11 years ago

Attachment: patch-204b.txt​ added

comment:5 by Yuri Dario, 11 years ago

Owner: changed from bird to Yuri Dario
Status: assigned → accepted

Added reworked patches for branch and trunk.

by Yuri Dario, 11 years ago

Attachment: patch-204t.txt​ added

comment:6 by bird, 11 years ago

Need to split up some bits there… let me have a go at it.

comment:7 by bird, 11 years ago

r3813, r3814 and r3816 implements chown, fchown and lchown on trunk. It should also improve *chmod operation slightly.

comment:8 by bird, 11 years ago

r3817 and r3818 makes an attempt at backporting it to 0.6. Haven't set up the necessary bits to build 0.6 here, so I don't know whether it really builds nor works.

comment:9 by bird, 11 years ago

Resolution: → fixed
Status: accepted → closed
Note: See TracTickets for help on using tickets.