<div dir="ltr"><div>Hi Partha,</div><div><br></div><div>First, thanks for the patch! Unfortunately, there are a couple of issues with this fix. Read on.</div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 20, 2015 at 2:59 PM, Partha Pratim Mukherjee <span dir="ltr"><<a href="mailto:ppm.floss@gmail.com" target="_blank">ppm.floss@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Destroy session command by default removes the default config file<br>
without checking the current session. As a result when we call any<br>
other command which expects a default session by calling<br>
get_session_name() function, it fails.<br>
<br>
This patch will fix this by checking that the default config file gets<br>
removed only when destroy session is called with the current session.<br>
<br>
Fixes: #887<br>
<br>
Signed-off-by: Partha Pratim Mukherjee <<a href="mailto:ppm.floss@gmail.com">ppm.floss@gmail.com</a>><br>
---<br>
 src/bin/lttng/commands/destroy.c |    4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/bin/lttng/commands/destroy.c b/src/bin/lttng/commands/destroy.c<br>
index 95343c9..3fcecc2 100644<br>
--- a/src/bin/lttng/commands/destroy.c<br>
+++ b/src/bin/lttng/commands/destroy.c<br>
@@ -90,7 +90,9 @@ static int destroy_session(struct lttng_session *session)<br>
        }<br>
<br>
        MSG("Session %s destroyed", session->name);<br>
-       config_destroy_default();<br>
+       if (strncmp(session->name, get_session_name(), NAME_MAX) == 0) {<br></blockquote><div><br></div><div><br></div><div>This will leak the string returned by get_session_name().</div><div><br></div><div>Moreover, config_read_session_name(), which is called by get_session_name() will spam the error logs anytime an .lttngrc file can't be found; the error logging statement should be moved out to the different callers.</div><div><br></div><div>Looking forward to a v2!</div><div>Jérémie<br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
+               config_destroy_default();<br>
+       }<br>
<br>
        if (lttng_opt_mi) {<br>
                ret = mi_lttng_session(writer, session, 0);<br>
<span class=""><font color="#888888">--<br>
1.7.10.4<br>
<br>
<br>
_______________________________________________<br>
lttng-dev mailing list<br>
<a href="mailto:lttng-dev@lists.lttng.org">lttng-dev@lists.lttng.org</a><br>
<a href="http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev" target="_blank">http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Jérémie Galarneau<br>EfficiOS Inc.<br><a href="http://www.efficios.com" target="_blank">http://www.efficios.com</a></div>
</div></div>