Mercurial > dovecot > core-2.2

changeset 19513:74850147fc47

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
lib: Avoid passing NULL to qsort() or bsearch() on empty array. This wouldn't normally matter because count=0 so it wouldn't be dereferenced anyway. But it triggered a run-time warning from clang, so lets fix it.
author Timo Sirainen <timo.sirainen@dovecot.fi>
date Mon, 14 Dec 2015 17:42:34 +0200
parents 5566fdec2593
children 9f3e9150b6a3
files src/lib/array.c
diffstat 1 files changed, 2 insertions(+), 0 deletions(-) [+]
line wrap: on
line diff
--- a/src/lib/array.c	Mon Dec 14 12:36:23 2015 +0200
+++ b/src/lib/array.c	Mon Dec 14 17:42:34 2015 +0200
@@ -124,6 +124,8 @@
 	unsigned int count;
 
 	count = array_count_i(array);
+	if (count == 0)
+		return;
 	qsort(buffer_get_modifiable_data(array->buffer, NULL),
 	      count, array->element_size, cmp);
 }