cmdline: free on exit

Malloc cl in the cmdline_stdin_new function, so release in the
cmdline_stdin_exit function is logical, so that cl will not be
released alone.

Fixes: af75078fece3 ("first public release")
Signed-off-by: Zhihong Peng <zhihongx.peng@intel.com>
Reviewed-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
Tested-by: Zhihong Peng <zhihongx.peng@intel.com>

diff --git a/app/test/test.c b/app/test/test.c
index 173d202e4774e55c400cd9974e0b05960167ccac..5194131026dab6003f97528b7bbd6e9cfd912264 100644 (file)
--- a/app/test/test.c
+++ b/app/test/test.c
@@ -233,7 +233,6 @@ main(int argc, char **argv)
 
                cmdline_interact(cl);
                cmdline_stdin_exit(cl);
-               cmdline_free(cl);
        }
 #endif
        ret = 0;

diff --git a/app/test/test_cmdline_lib.c b/app/test/test_cmdline_lib.c
index 054ebf5e9dc1f70f4b57ae636e0b79c42152ab1c..f238094b075029f3ec8d781f7ed876a61cd3f3b3 100644 (file)
--- a/app/test/test_cmdline_lib.c
+++ b/app/test/test_cmdline_lib.c
@@ -180,7 +180,6 @@ test_cmdline_socket_fns(void)
        /* void functions */
        cmdline_stdin_exit(NULL);
 
-       cmdline_free(cl);
        return 0;
 error:
        printf("Error: function accepted null parameter!\n");

diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
index 9b1843eb098ec1dfe048f6dce20d7985fd11afea..450bcbb445bc0570dd1ea8ed085c51a44fb2d921 100644 (file)
--- a/doc/guides/rel_notes/release_21_11.rst
+++ b/doc/guides/rel_notes/release_21_11.rst
@@ -325,6 +325,9 @@ API Changes
   removed. Its usages have been replaced by a new function
   ``rte_kvargs_get_with_value()``.
 
+* cmdline: ``cmdline_stdin_exit()`` now frees the ``cmdline`` structure.
+  Calls to ``cmdline_free()`` after it need to be deleted from applications.
+
 * cmdline: Made ``cmdline`` structure definition hidden on Linux and FreeBSD.
 
 * cmdline: Made ``rdline`` structure definition hidden. Functions are added

diff --git a/lib/cmdline/cmdline_socket.c b/lib/cmdline/cmdline_socket.c
index 998e8ade25c1e56886c17ce7205733c0ea9c829a..ebd5343754544ba038df8d719cde331ab1da4556 100644 (file)
--- a/lib/cmdline/cmdline_socket.c
+++ b/lib/cmdline/cmdline_socket.c
@@ -53,4 +53,5 @@ cmdline_stdin_exit(struct cmdline *cl)
                return;
 
        terminal_restore(cl);
+       cmdline_free(cl);
 }