Bug 855416
| Summary: | regression: chown does not work in NFS4 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 6 | Reporter: | RHEL Program Management <pm-rhel> | ||||
| Component: | nfs-utils | Assignee: | Steve Dickson <steved> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | JianHong Yin <jiyin> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | urgent | ||||||
| Version: | 6.3 | CC: | eguan, jiyin, jkurik, jsvarova, jwest, pm-eus, rh-bugzilla, steved, yanwang | ||||
| Target Milestone: | rc | Keywords: | Regression, ZStream | ||||
| Target Release: | --- | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | nfs-utils-1.2.3-15.el6_2.2 | Doc Type: | Bug Fix | ||||
| Doc Text: |
Due to a regression, the NFSv4 idmapper utility did not initialize correctly, which lead to a creation of incorrect file permissions. With this update, idmapper initializes as expected, creating correct file permissions.
|
Story Points: | --- | ||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2014-01-15 12:29:52 UTC | Type: | --- | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | 849945, 1020653 | ||||||
| Bug Blocks: | 823848 | ||||||
| Attachments: |
|
||||||
|
Description
RHEL Program Management
2012-09-07 16:40:04 UTC
Created attachment 625074 [details]
The patch that fixes the problem
verify by : https://beaker.engineering.redhat.com/jobs/524526 6.2 test PASS https://beaker.engineering.redhat.com/jobs/524558 6.2 + errata, test FAIL; :: [ FAIL ] :: Test root hasn't privilege to chown its file since squash to nobody (Expected 1, got 0) ---------------------------------------------------------------- 6.2 is OK, why backported to 6.2 z-stream? Hi all
this bug is not in RHEL6.2, no need a zstream errata for 6.2;
I think we should close this bug, and delete the errata.
(In reply to Yin.JianHong from comment #11) > Hi all > this bug is not in RHEL6.2, no need a zstream errata for 6.2; > I think we should close this bug, and delete the errata. Here is the patch that went in: diff -up nfs-utils-1.2.3/utils/idmapd/idmapd.c.orig nfs-utils-1.2.3/utils/idmapd/idmapd.c --- nfs-utils-1.2.3/utils/idmapd/idmapd.c.orig 2012-10-10 11:23:46.229652000 -0400 +++ nfs-utils-1.2.3/utils/idmapd/idmapd.c 2012-10-10 13:36:31.164381000 -0400 @@ -836,7 +836,7 @@ nametoidres(struct idmap_msg *im) switch (im->im_type) { case IDMAP_TYPE_USER: - ret = nfs4_name_to_uid(im->im_name, &uid); + ret = nfs4_owner_to_uid(im->im_name, &uid); im->im_id = (u_int32_t) uid; if (ret) { im->im_status = IDMAP_STATUS_LOOKUPFAIL; @@ -844,7 +844,7 @@ nametoidres(struct idmap_msg *im) } return; case IDMAP_TYPE_GROUP: - ret = nfs4_name_to_gid(im->im_name, &gid); + ret = nfs4_group_owner_to_gid(im->im_name, &gid); im->im_id = (u_int32_t) gid; if (ret) { im->im_status = IDMAP_STATUS_LOOKUPFAIL; This allows clients to understand uid/gid strings that are returned by the server in both the "3606" format and "steved" format. (In reply to Yin.JianHong from comment #11) > Hi all > this bug is not in RHEL6.2, no need a zstream errata for 6.2; > I think we should close this bug, and delete the errata. Checked with Steve Dickson - I am closing the bug and removing the errata. |