Re: virtualbox /proc/modules warning

Alex Efros <[email protected]>
Newsgroups gmane.linux.gentoo.hardened
Organization http://powerman.name/
Message-ID <[email protected]>
Hi!

On Sat, Sep 14, 2013 at 12:16:20AM +0400, Peter Volkov wrote:
> There are other use-cases where this warning is useless, like running
> virtualbox inside openvz container :) /usr/bin/VirtualBox binary is a
> shell script that you can easy modify to avoid this warning. That said I
> don't think it's worth to have such patch in portage tree - just apply
> it locally.

Why is that? How such patch may harm anything?

--- /opt/VirtualBox/VBox.sh.old	2013-09-14 00:45:13.873137602 +0300
+++ /opt/VirtualBox/VBox.sh	2013-09-14 00:45:17.553137565 +0300
@@ -32,7 +32,7 @@
 
 if [ "$1" = "shutdown" ]; then
     SHUTDOWN="true"
-elif ! lsmod|grep -q vboxdrv; then
+elif [ -r /proc/modules ] && ! lsmod|grep -q vboxdrv; then
     cat << EOF
 WARNING: The VirtualBox kernel modules are not loaded.
          Please load all the needed kernel modules by:


-- 
			WBR, Alex.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.