Mercurial > dropbear
annotate fuzz/fuzzer-kexecdh.c @ 1790:42745af83b7d
Introduce extra delay before closing unauthenticated sessions
To make it harder for attackers, introduce a delay to keep an
unauthenticated session open a bit longer, thus blocking a connection
slot until after the delay.
Without this, while there is a limit on the amount of attempts an attacker
can make at the same time (MAX_UNAUTH_PER_IP), the time taken by dropbear to
handle one attempt is still short and thus for each of the allowed parallel
attempts many attempts can be chained one after the other. The attempt rate
is then:
"MAX_UNAUTH_PER_IP / <process time of one attempt>".
With the delay, this rate becomes:
"MAX_UNAUTH_PER_IP / UNAUTH_CLOSE_DELAY".
author | Thomas De Schampheleire <thomas.de_schampheleire@nokia.com> |
---|---|
date | Wed, 15 Feb 2017 13:53:04 +0100 |
parents | 0cc85b4a4abb |
children |
rev | line source |
---|---|
1589 | 1 #include "fuzz.h" |
2 #include "session.h" | |
3 #include "fuzz-wrapfd.h" | |
4 #include "debug.h" | |
5 #include "runopts.h" | |
6 #include "algo.h" | |
7 #include "bignum.h" | |
8 | |
1772
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
9 static const struct dropbear_kex *ecdh[3]; /* 256, 384, 521 */ |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
10 static struct key_context* keep_newkeys = NULL; |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
11 /* number of generated parameters. An arbitrary limit, but will delay startup */ |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
12 #define NUM_PARAMS 80 |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
13 static struct kex_ecdh_param *ecdh_params[NUM_PARAMS]; |
1589 | 14 |
1772
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
15 static void setup() __attribute__((constructor)); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
16 // Perform initial setup here to avoid hitting timeouts on first run |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
17 static void setup() { |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
18 fuzz_common_setup(); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
19 fuzz_svr_setup(); |
1589 | 20 |
1772
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
21 /* ses gets zeroed by fuzz_set_input */ |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
22 keep_newkeys = (struct key_context*)m_malloc(sizeof(struct key_context)); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
23 ecdh[0] = fuzz_get_algo(sshkex, "ecdh-sha2-nistp256"); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
24 ecdh[1] = fuzz_get_algo(sshkex, "ecdh-sha2-nistp384"); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
25 ecdh[2] = fuzz_get_algo(sshkex, "ecdh-sha2-nistp521"); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
26 assert(ecdh[0]); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
27 assert(ecdh[1]); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
28 assert(ecdh[2]); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
29 keep_newkeys->algo_hostkey = DROPBEAR_SIGNKEY_ECDSA_NISTP256; |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
30 ses.newkeys = keep_newkeys; |
1589 | 31 |
1772
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
32 /* Pre-generate parameters */ |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
33 int i; |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
34 for (i = 0; i < NUM_PARAMS; i++) { |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
35 ses.newkeys->algo_kex = ecdh[i % 3]; |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
36 ecdh_params[i] = gen_kexecdh_param(); |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
37 } |
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
38 } |
1589 | 39 |
1772
0cc85b4a4abb
Move fuzzer-kex initialisation into a constructor function
Matt Johnston <matt@ucc.asn.au>
parents:
1756
diff
changeset
|
40 int LLVMFuzzerTestOneInput(const uint8_t *Data, size_t Size) { |
1589 | 41 |
42 if (fuzz_set_input(Data, Size) == DROPBEAR_FAILURE) { | |
43 return 0; | |
44 } | |
45 | |
46 m_malloc_set_epoch(1); | |
47 | |
48 if (setjmp(fuzz.jmp) == 0) { | |
49 /* Based on recv_msg_kexdh_init()/send_msg_kexdh_reply() | |
50 with DROPBEAR_KEX_ECDH */ | |
51 ses.newkeys = keep_newkeys; | |
52 | |
53 /* random choice of ecdh 256, 384, 521 */ | |
54 unsigned char b = buf_getbyte(fuzz.input); | |
55 ses.newkeys->algo_kex = ecdh[b % 3]; | |
56 | |
57 /* Choose from the collection of ecdh params */ | |
58 unsigned int e = buf_getint(fuzz.input); | |
59 struct kex_ecdh_param *ecdh_param = ecdh_params[e % NUM_PARAMS]; | |
60 | |
61 buffer * ecdh_qs = buf_getstringbuf(fuzz.input); | |
62 | |
1606
98d2b125eb89
kexhashbuf was much to small in kex fuzzers
Matt Johnston <matt@ucc.asn.au>
parents:
1595
diff
changeset
|
63 ses.kexhashbuf = buf_new(KEXHASHBUF_MAX_INTS); |
1589 | 64 kexecdh_comb_key(ecdh_param, ecdh_qs, svr_opts.hostkey); |
65 | |
1609 | 66 mp_clear(ses.dh_K); |
1589 | 67 m_free(ses.dh_K); |
68 buf_free(ecdh_qs); | |
69 | |
1609 | 70 buf_free(ses.hash); |
71 buf_free(ses.session_id); | |
72 /* kexhashbuf is freed in kexdh_comb_key */ | |
73 | |
1589 | 74 m_malloc_free_epoch(1, 0); |
75 } else { | |
76 m_malloc_free_epoch(1, 1); | |
77 TRACE(("dropbear_exit longjmped")) | |
78 /* dropbear_exit jumped here */ | |
79 } | |
80 | |
81 return 0; | |
82 } |