-
-
Notifications
You must be signed in to change notification settings - Fork 75
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
aborts when no modules present #138
Comments
Hey, thanks for reporting!
Yes I agree. Would you be interested in submitting a PR to fix this? |
hey, @orhun. I tried changing that mentioned line to something like this: let modules_content = util::exec_cmd("sh", &["-c", &module_read_cmd]).unwrap_or_else(|_| String::new()); should I also add few lines for parsing part? |
Would also probably be worth adding a log message or something since the display is just blank |
Thanks, saves me the time doing an MR lol🙏 |
in this case under lxc on a chromebook (and probably on generic linux in a few other cases eg with no runtime modules) there is no
/proc/modules
, so this panic's. In this case could it just return and empty string so it would not panic, and then show an empty set of modules?kmon/src/kernel/lkm.rs
Lines 117 to 118 in 3ed516f
The text was updated successfully, but these errors were encountered: