On Mon, Jan 21, 2019 at 08:44:48PM +0100, Greg KH wrote:
> On Mon, Jan 21, 2019 at 07:35:43PM +0000, Alakesh Haloi wrote:
> > Hello Greg,
> > Can you please consider including the following patch in the stable
> > linux-4.14.y branch? This is to fix a broken dependency for config option
> > CONFIG_CIFS_POSIX
> >
> > 04f67ed3d83a ("cifs: allow disabling insecure dialects in the config")
>
> I do not see that commit in Linus's tree, are you sure the sha1 is
> correct?
>
Apologies for pasting a wrong sha1 for the commit. Following is the commit from
linus's tree that needs to be included to fix the dependency:
7420451f6a10 ("cifs: allow disabling insecure dialects in the config")
> > Following commit introduced a dependency for CONFIG_CIFS_POSIX on the config
> > CONFIG_CIFS_ALLOW_INSECURE_LEGACY, but the above mentioned commit that had
> > enable CONFIG_CIFS_POSIX in 4.14.94.
> >
> > ab9697222a1d ("cifs: In Kconfig CONFIG_CIFS_POSIX needs depends on legacy
> > (insecure cifs)")
>
> Was the above patch incorrectly applied? Should it just be dropped
> instead?
This patch that is already there in 4.14.y looks correct but it does
have missing dependency on the patch that is requested to be included.
The other option is to drop this patch. But I would prefer to keep the
patch and have the dependency resolved.
>
> thanks,
>
> greg k-h
