[ltt-dev] [UST PATCH] ustctl: Fix memory allocation problem with compatibility args

Jason Wessel jason.wessel at windriver.com
Wed Mar 2 16:16:09 EST 2011


The parenthesis were missing to make the malloc math have the correct
precedence.  The addition needs to occur before the multiplication.
The result is the same but for clarity also change change char ** to
char * because we are allocating an array of char pointers.

Signed-off-by: Jason Wessel <jason.wessel at windriver.com>
---
 ustctl/ustctl.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ustctl/ustctl.c b/ustctl/ustctl.c
index d642178..807af8c 100644
--- a/ustctl/ustctl.c
+++ b/ustctl/ustctl.c
@@ -121,7 +121,7 @@ int main(int argc, char *argv[])
 		case DESTROY_TRACE:
 		case ENABLE_MARKER:
 		case DISABLE_MARKER:
-			args = (char **)malloc(sizeof(char **) * argc + 3);
+			args = (char **)malloc(sizeof(char *) * (argc + 3));
 			optind--;
 			args[optind] = strdup(&argv[optind][2]);
 			for (i = optind + 1; i < argc; i++) {
-- 
1.6.6.2





More information about the lttng-dev mailing list