diff options
| author | Christophe Fergeau <cfergeau@redhat.com> | 2012-09-11 18:03:24 +0200 |
|---|---|---|
| committer | Christophe Fergeau <cfergeau@redhat.com> | 2012-09-20 16:40:54 +0200 |
| commit | eb19ac081f66d72f88868bb977c2b7e4a2b0259b (patch) | |
| tree | d6f178a67008c0e0360d46d3a835c64cb1668a29 /server/reds.c | |
| parent | dc3cd205dda4d76f4afecd64d4d5abde81545e6d (diff) | |
| download | spice-eb19ac081f66d72f88868bb977c2b7e4a2b0259b.tar.gz spice-eb19ac081f66d72f88868bb977c2b7e4a2b0259b.tar.xz spice-eb19ac081f66d72f88868bb977c2b7e4a2b0259b.zip | |
reds: Abort on BN-new failures
BN_new returns NULL on allocation failures. Given that we abort
on malloc allocation failures, we should also abort here. The
current code will segfault when BN_new fails as it immediatly tries
to use the NULL pointer.
Diffstat (limited to 'server/reds.c')
| -rw-r--r-- | server/reds.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/server/reds.c b/server/reds.c index 5537c159..4dd713e0 100644 --- a/server/reds.c +++ b/server/reds.c @@ -1859,7 +1859,7 @@ static void openssl_init(RedLinkInfo *link) link->tiTicketing.bn = BN_new(); if (!link->tiTicketing.bn) { - spice_warning("OpenSSL BIGNUMS alloc failed"); + spice_error("OpenSSL BIGNUMS alloc failed"); } BN_set_word(link->tiTicketing.bn, f4); |