auth: token jar handling should realloc correctly
authorDerrick Brashear <shadow@your-file-system.com>
Wed, 26 Sep 2012 11:37:11 +0000 (07:37 -0400)
committerDerrick Brashear <shadow@your-file-system.com>
Wed, 26 Sep 2012 12:28:11 +0000 (05:28 -0700)
instead of reallocing n+1 tokens, we would realloc 1 token
plus N bytes

Change-Id: Idb5a20b5b9becc9d48cb645bbc0bd26459220ccd
Reviewed-on: http://gerrit.openafs.org/8163
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>

src/auth/token.c

index 9d99a1e..5e10543 100644 (file)
@@ -174,7 +174,7 @@ addOpaque(struct ktc_setTokenData *jar, char *data, size_t len)
 
     entry = jar->tokens.tokens_len;
     jar->tokens.tokens_val = realloc(jar->tokens.tokens_val,
-                                    entry + 1 * sizeof(token_opaque));
+                                    (entry + 1) * sizeof(token_opaque));
     jar->tokens.tokens_len++;
     jar->tokens.tokens_val[entry].token_opaque_val = data;
     jar->tokens.tokens_val[entry].token_opaque_len = len;