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

official debian package #218

Closed
alexmyczko opened this issue Nov 20, 2021 · 18 comments
Closed

official debian package #218

alexmyczko opened this issue Nov 20, 2021 · 18 comments

Comments

@alexmyczko
Copy link

it got rejected for:

I also wonder whether a software that uses md4 shall be added to the archive ...

@namjaejeon
Copy link
Member

Hm.. What is problem using md4 ?

@alexmyczko
Copy link
Author

alexmyczko commented Nov 21, 2021

or according to https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=561a1cf57535154f094f31167a9170197caae686

is it still needed?

i can only see it used for ksmbd.adduser, but i'm not expert enough to say it's still needed/useful or not

@namjaejeon
Copy link
Member

or according to https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=561a1cf57535154f094f31167a9170197caae686

This patch is to remove md4 use in ksmbd kernel module, not add it. ksmbd-tools is not related with this one. And ksmbd-tools included ksmbd.mountd as well as ksmbd.adduser. If there is no ksmbd.mountd, user can not run ksmbd server. User need to launch ksmbd.mountd in ksmbd-tools to run ksmbd kernel server.

@alexmyczko
Copy link
Author

alexmyczko commented Nov 22, 2021

so without md4 it will not work right? [Y/n]

@hclee
Copy link
Member

hclee commented Nov 23, 2021

@alexmyczko

so without md4 it will not work right? [Y/n]

Yes, md4 is needed.

@namjaejeon
Copy link
Member

@alexmyczko without md4, ksmbd will work fine. ksmbd no longer need md4.

@namjaejeon
Copy link
Member

@alexmyczko ksmbd I said is ksmbd in mainline kernel, not out of tree ksmbd at github. out of tree ksmbd contain smb1 protocol suppprt, it need md4 library in kernel.

@alexmyczko
Copy link
Author

so if i wanted to disable smb1 support i can get rid of md4?

@namjaejeon
Copy link
Member

I would like to know what ksmbd is used ? ksmbd in mainline kernel or ksmbd in this github ?

@namjaejeon
Copy link
Member

ksmbd in mainline kernel doesn't support SMB1.

@alexmyczko
Copy link
Author

alexmyczko commented Nov 23, 2021

of course i mean ksmbd-tools where the issue is filed at sorry for the confusion

maybe this could be made a build time option, but i'd need to repack the source package for debian without the option to support SMB1/NTLM1

@namjaejeon
Copy link
Member

Ah.. ksmbd and ksmbd-tools are different... if you are saying ksmbd-tools, not ksmbd kernel module for md4, I have to change my words. ksmbd-tools contain own md4 library.
Could you please tell me why you want to remove md4 for packaging in debian ?

@alexmyczko
Copy link
Author

https://en.wikipedia.org/wiki/MD4?wprov=sfti1

I am not the who wants to remove it, they do not want it in Debian for good reasons (security):
https://ftp-master.debian.org

@namjaejeon
Copy link
Member

Okay, Samba must also need md4 hash for NT password encryption. cifs client kernel module also. It makes no sense that samba is fine and ksmbd is not for md4 use.

@namjaejeon
Copy link
Member

Please read this discussion threads : https://www.spinics.net/lists/linux-cifs/msg22555.html

@namjaejeon
Copy link
Member

namjaejeon commented Nov 28, 2021

Could you please share what's going on ?

@alexmyczko
Copy link
Author

all fine, let's see 2nd round of NEW queue... (i had the license wrong for md4* files)

@alexmyczko
Copy link
Author

it just entered sid!

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

No branches or pull requests

3 participants