diff options
| author | Nick Mathewson <nickm@torproject.org> | 2011-11-07 10:44:02 -0500 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2011-11-07 10:44:02 -0500 |
| commit | 51f53b590ee15c972adcff9e7d94c904a9267ac2 (patch) | |
| tree | eb0d3809d3eeb64056538bbf60a745c0a0465eab | |
| parent | 350fe952590fdfd1711aaaddcb478cdea20b6e6a (diff) | |
| parent | 4d8306e0e92200e1a25a073480ec342e22e590b7 (diff) | |
| download | tor-51f53b590ee15c972adcff9e7d94c904a9267ac2.tar.gz tor-51f53b590ee15c972adcff9e7d94c904a9267ac2.zip | |
Merge remote-tracking branch 'origin/maint-0.2.2'
| -rw-r--r-- | changes/bug4410 | 5 | ||||
| -rw-r--r-- | src/or/routerparse.c | 5 |
2 files changed, 10 insertions, 0 deletions
diff --git a/changes/bug4410 b/changes/bug4410 new file mode 100644 index 0000000000..f42893adf4 --- /dev/null +++ b/changes/bug4410 @@ -0,0 +1,5 @@ + o Major bugfixes: + - Correctly sanity-check that we don't underflow on a memory allocation + for introduction point decryption. Bug discovered by Dan Rosenberg. + Fixes bug 4410; bugfix on 0.2.1.5-alpha. + diff --git a/src/or/routerparse.c b/src/or/routerparse.c index e8b2dd7d2b..4ea7b964cf 100644 --- a/src/or/routerparse.c +++ b/src/or/routerparse.c @@ -4886,6 +4886,11 @@ rend_decrypt_introduction_points(char **ipos_decrypted, crypto_cipher_env_t *cipher; char *dec; int declen; + if (ipos_encrypted_size < CIPHER_IV_LEN + 2) { + log_warn(LD_REND, "Size of encrypted introduction points is too " + "small."); + return -1; + } dec = tor_malloc_zero(ipos_encrypted_size - CIPHER_IV_LEN - 1); cipher = crypto_create_init_cipher(descriptor_cookie, 0); declen = crypto_cipher_decrypt_with_iv(cipher, dec, |