dovecot-2.2: lib: data-stack - disambiguate sizes in t_pop_verify

dovecot at dovecot.org dovecot at dovecot.org
Mon Jul 28 13:54:28 UTC 2014


details:   http://hg.dovecot.org/dovecot-2.2/rev/9d85b31dfb74
changeset: 17642:9d85b31dfb74
user:      Phil Carmody <phil at dovecot.fi>
date:      Mon Jul 28 16:45:33 2014 +0300
description:
lib: data-stack - disambiguate sizes in t_pop_verify
In DEBUG mode, the allocated size is bigger than the requested size, so
rename the variable to reflect its real meaning, and move it into a
tighter scope in the process.

Signed-off-by: Phil Carmody <phil at dovecot.fi>

diffstat:

 src/lib/data-stack.c |  12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diffs (31 lines):

diff -r 1778c2e77cfa -r 9d85b31dfb74 src/lib/data-stack.c
--- a/src/lib/data-stack.c	Mon Jul 28 16:45:33 2014 +0300
+++ b/src/lib/data-stack.c	Mon Jul 28 16:45:33 2014 +0300
@@ -223,7 +223,7 @@
 {
 	struct stack_block *block;
 	unsigned char *p;
-	size_t pos, max_pos, used_size, alloc_size;
+	size_t pos, max_pos, used_size;
 
 	block = current_frame_block->block[frame_pos];
 	pos = block->size - current_frame_block->block_space_used[frame_pos];
@@ -232,13 +232,13 @@
 		used_size = block->size - block->left;
 		p = STACK_BLOCK_DATA(block);
 		while (pos < used_size) {
-			alloc_size = *(size_t *)(p + pos);
-			if (used_size - pos < alloc_size)
+			size_t requested_size = *(size_t *)(p + pos);
+			if (used_size - pos < requested_size)
 				i_panic("data stack[%s]: saved alloc size broken",
 					current_frame_block->marker[frame_pos]);
-			pos += MEM_ALIGN(sizeof(alloc_size));
-			max_pos = pos + MEM_ALIGN(alloc_size + SENTRY_COUNT);
-			pos += alloc_size;
+			pos += MEM_ALIGN(sizeof(size_t));
+			max_pos = pos + MEM_ALIGN(requested_size + SENTRY_COUNT);
+			pos += requested_size;
 
 			for (; pos < max_pos; pos++) {
 				if (p[pos] != CLEAR_CHR)


More information about the dovecot-cvs mailing list