From 1c87a27574023096212375329e5c4ba1d82f71ce Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Tue, 15 Dec 2009 15:51:59 -0500 Subject: [PATCH] Fix bug 1173: remove an assert(unsigned >= 0). --- ChangeLog | 2 ++ src/common/crypto.c | 1 - 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 22da2f70b1..9f7bc41ecb 100644 --- a/ChangeLog +++ b/ChangeLog @@ -16,6 +16,8 @@ Changes in Version 0.2.1.21 - 20??-??-?? - Fix a couple of very-hard-to-trigger memory leaks, and one hard-to- trigger platform-specific option misparsing case found by Coverity Scan. + - Fix a compilation warning on Fedora 12 by removing an impossible-to- + trigger assert. Fixes bug 1173. Changes in version 0.2.1.20 - 2009-10-15 diff --git a/src/common/crypto.c b/src/common/crypto.c index da38ddc62e..8676b6bc7b 100644 --- a/src/common/crypto.c +++ b/src/common/crypto.c @@ -532,7 +532,6 @@ crypto_pk_write_key_to_string_impl(crypto_pk_env_t *env, char **dest, (void)BIO_set_close(b, BIO_NOCLOSE); /* so BIO_free doesn't free buf */ BIO_free(b); - tor_assert(buf->length >= 0); *dest = tor_malloc(buf->length+1); memcpy(*dest, buf->data, buf->length); (*dest)[buf->length] = 0; /* nul terminate it */