Bug 14742 - Calling Windows info levels with SMB1trans2:setpathinfo with a POSIX connection on symlinks can crash smbd.
Summary: Calling Windows info levels with SMB1trans2:setpathinfo with a POSIX connecti...
Status: ASSIGNED
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: File services (show other bugs)
Version: 4.14.5
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-15 23:24 UTC by Jeremy Allison
Modified: 2021-07-11 15:23 UTC (History)
2 users (show)

See Also:


Attachments
git-am fix for 4.14.next. (10.93 KB, patch)
2021-06-16 17:49 UTC, Jeremy Allison
slow: review-
Details
git-am fix for 4.14.next. (10.98 KB, patch)
2021-06-16 18:27 UTC, Jeremy Allison
slow: review+
npower: review+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Allison 2021-06-15 23:24:38 UTC

    
Comment 1 Jeremy Allison 2021-06-15 23:24:54 UTC
Have path, need bugnumber.
Comment 2 Samba QA Contact 2021-06-16 11:58:03 UTC
This bug was referenced in samba master:

ac10058d7f6b4605157f508189a448310f5f18da
263c95aee38c9198ad9a30c4d960d72f46b7c27a
Comment 3 Jeremy Allison 2021-06-16 17:49:37 UTC
Created attachment 16662 [details]
git-am fix for 4.14.next.
Comment 4 Jeremy Allison 2021-06-16 18:12:55 UTC
NB. I checked and POSIX-SYMLINK-SETPATHINFO run from master passes against 4.13, so this is something we only recently broke with the smb_fname->fsp changes I think (thank goodness).

So we only need this for 4.14.next.

Thanks Noel !
Comment 5 Ralph Böhme 2021-06-16 18:23:54 UTC
Comment on attachment 16662 [details]
git-am fix for 4.14.next.

Patch 1/2 is missing reviewed-by. Can we fix this at least for the backport?
Comment 6 Jeremy Allison 2021-06-16 18:27:07 UTC
Created attachment 16663 [details]
git-am fix for 4.14.next.

Updated with added RB. Sorry for missing that.
Comment 7 Ralph Böhme 2021-06-16 19:20:56 UTC
Reassigning to Karolin for inclusion in 4.14.
Comment 8 Karolin Seeger 2021-06-29 06:47:07 UTC
(In reply to Ralph Böhme from comment #7)
Pushed to autobuild-v4-14-test.