libkmod: reset was_space on second pass

The softdep config parser uses a 2-pass approach to use a single
allocation for all the softdep struct. However "was_space" variable
isn't reset between them. This can lead to a buffer overflow.

Reported-by: Jorge Lucangeli Obes <jorgelo@google.com>
Link: https://lore.kernel.org/linux-modules/CAKYuF5QhGCPCazHQjN-=kFc5kHs7Ok8WqmmGLo31CiOEN8TYdA@mail.gmail.com

diff --git a/libkmod/libkmod-config.c b/libkmod/libkmod-config.c
index aaac0a1393c28ea6dbc309e2c440c92a44eff6a9..7b6236779c13fc859e277ae2f3fd7616ee87313e 100644 (file)
--- a/libkmod/libkmod-config.c
+++ b/libkmod/libkmod-config.c
@@ -335,6 +335,7 @@ static int kmod_config_add_softdep(struct kmod_config *config,
        n_pre = 0;
        n_post = 0;
        mode = S_NONE;
+       was_space = false;
        for (p = s = line; ; s++) {
                size_t plen;