Merge lp://qastaging/~csurbhi/ubuntu/maverick/samba/samba-fix.276472 into lp://qastaging/ubuntu/maverick/samba

Proposed by Surbhi Palande
Status: Merged
Merge reported by: Steve Langasek
Merged at revision: not available
Proposed branch: lp://qastaging/~csurbhi/ubuntu/maverick/samba/samba-fix.276472
Merge into: lp://qastaging/ubuntu/maverick/samba
Diff against target: 67 lines (+28/-1)
5 files modified
.pc/applied-patches (+1/-0)
debian/changelog (+8/-0)
debian/patches/error-trans.fix-276472 (+15/-0)
debian/patches/series (+1/-0)
source3/lib/errmap_unix.c (+3/-1)
To merge this branch: bzr merge lp://qastaging/~csurbhi/ubuntu/maverick/samba/samba-fix.276472
Reviewer Review Type Date Requested Status
Steve Langasek Needs Information
Ubuntu branches Pending
Review via email: mp+32753@code.qastaging.launchpad.net

Description of the change

Adds the translation of Unix Error code -ENOTSUP to NT Error Code NT_STATUS_NOT_SUPPORTED. The absence of this translation wrongly sends back to the client a STATUS_DENIED message.

I have lightly tested this fix on my local machine and also asked users to test it.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

The change makes sense, but should be reviewed by protocol experts before we commit to it. Please submit this patch to <email address hidden> for review; once it's been submitted upstream, I'm happy to apply it to the package.

review: Needs Information

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: