Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge fixes for updated Dom0 kernel #429

Merged
merged 3 commits into from
Feb 6, 2019
Merged

Conversation

MarkSymsCtx
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Feb 6, 2019

Coverage Status

Coverage increased (+0.002%) to 33.259% when pulling 1d3f7d1 on team/ring0/linux-upstream into 46a8c7a on master.

anoobs and others added 3 commits February 6, 2019 10:27
With pre-4.14 kernel, if the security type specified using "sec=" mount option
is not supported, the SMB2 session setup code changes the security type
to RawNTLMSSP. This behaviour has been changed in 4.14.y, where it would now
fail the mount and return an error, if "sec=" mount option is not supported.

SMB2 session setup code now uses the global security flags
(/proc/fs/cifs/SecurityFlags) to select the security method to be used when
no security method is specified.

Signed-off-by: Mark Syms <mark.syms@citrix.com>
Signed-off-by: Anoob Soman <anoob.soman@citrix.com>
In Linux 4.14 cifs module has soft dependencies (pre) on crypto modules. This
soft dependencies makes kmod (modprobe) ignore commands inside
/etc/modprobe.d/cifs.conf. cifs.conf contains command to change default value
of SecurityFlags from 0x85 to 0x83. Since the commands inside cifs.conf are
ignored while cifs module is loaded, SecurityFlags remains at 0x85, breaking
few tests.

Remove cifs.conf and change default value of SecurityFlags (by adding a kernel
patch) to 0x83

Signed-off-by: Mark Syms <mark.syms@citrix.com>
Signed-off-by: Anoob Soman <anoob.soman@citrix.com>
Signed-off-by: Mark Syms <mark.syms@citrix.com>
Copy link
Contributor

@chandrikas chandrikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MarkSymsCtx MarkSymsCtx merged commit 0357016 into master Feb 6, 2019
@MarkSymsCtx MarkSymsCtx deleted the team/ring0/linux-upstream branch August 20, 2019 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants