On Wed, Mar 6, 2019 at 1:43 PM Tobin C. Harding <tobin@xxxxxxxxxx> wrote: > > Currently the test_printf module does not have an exit function, this > prevents the module from being unloaded. If we cannot unload the > module we cannot run the tests a second time. > > Add an empty exit function. > > Signed-off-by: Tobin C. Harding <tobin@xxxxxxxxxx> Acked-by: Kees Cook <keescook@xxxxxxxxxxxx> -Kees > --- > lib/test_printf.c | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/lib/test_printf.c b/lib/test_printf.c > index 659b6cc0d483..601e8519319a 100644 > --- a/lib/test_printf.c > +++ b/lib/test_printf.c > @@ -615,5 +615,11 @@ test_printf_init(void) > > module_init(test_printf_init); > > +static void __exit test_printf_exit(void) > +{ > +} > + > +module_exit(test_printf_exit); > + > MODULE_AUTHOR("Rasmus Villemoes <linux@xxxxxxxxxxxxxxxxxx>"); > MODULE_LICENSE("GPL"); > -- > 2.20.1 > -- Kees Cook