diff mbox

[lttng-tools] Fix: add missing refcount of loaded modules

Message ID 1479749760-20017-1-git-send-email-francis.deslauriers@efficios.com
State Accepted, archived
Headers show

Commit Message

Francis Deslauriers Nov. 21, 2016, 5:36 p.m. UTC
Signed-off-by: Francis Deslauriers <francis.deslauriers at efficios.com>
---
 src/bin/lttng-sessiond/modprobe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jérémie Galarneau Nov. 21, 2016, 5:56 p.m. UTC | #1
Good catch! Merged in master, stable-2.9 and stable-2.8.

Thanks!
Jérémie



On 21 November 2016 at 12:36, Francis Deslauriers
<francis.deslauriers at efficios.com> wrote:
> Signed-off-by: Francis Deslauriers <francis.deslauriers at efficios.com>
> ---
>  src/bin/lttng-sessiond/modprobe.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c
> index 0502740..03545ef 100644
> --- a/src/bin/lttng-sessiond/modprobe.c
> +++ b/src/bin/lttng-sessiond/modprobe.c
> @@ -343,7 +343,7 @@ static int modprobe_lttng(struct kern_modules_param *modules,
>                         }
>                 } else {
>                         DBG("Modprobe successfully %s", modules[i].name);
> -                       modules[i].loaded;
> +                       modules[i].loaded = true;
>                 }
>         }
>
> --
> 2.7.4
>
diff mbox

Patch

diff --git a/src/bin/lttng-sessiond/modprobe.c b/src/bin/lttng-sessiond/modprobe.c
index 0502740..03545ef 100644
--- a/src/bin/lttng-sessiond/modprobe.c
+++ b/src/bin/lttng-sessiond/modprobe.c
@@ -343,7 +343,7 @@  static int modprobe_lttng(struct kern_modules_param *modules,
 			}
 		} else {
 			DBG("Modprobe successfully %s", modules[i].name);
-			modules[i].loaded;
+			modules[i].loaded = true;
 		}
 	}