diff options
| author | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2012-02-11 02:09:20 +0000 |
|---|---|---|
| committer | Ronnie Sahlberg <ronnie_sahlberg@ozemail.com.au> | 2012-02-11 02:09:20 +0000 |
| commit | 4d2cb8bf94776c1f8386fdb95ad0d50ecdea86f4 (patch) | |
| tree | 5febd98bb613c5c41e72da910c85f855e1be0f27 | |
| parent | 7a8e42888b49f9b76a31ba3059e6bc0c6327e704 (diff) | |
NETLOGON: Fix bugs with netlogon decryption not working in some cases
If there is only one Authenticatex in the file, decryption is not
working.
from Matthieu Patou <mat@matws.net>
svn path=/trunk/; revision=40960
| -rw-r--r-- | epan/dissectors/packet-dcerpc-netlogon.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/epan/dissectors/packet-dcerpc-netlogon.c b/epan/dissectors/packet-dcerpc-netlogon.c index c75ca45903..6af34dd286 100644 --- a/epan/dissectors/packet-dcerpc-netlogon.c +++ b/epan/dissectors/packet-dcerpc-netlogon.c @@ -2538,11 +2538,12 @@ netlogon_dissect_netrserverreqchallenge_rqst(tvbuff_t *tvb, int offset, g_hash_table_insert(netlogon_auths, key, vars); } else { - while(existing_vars->next != NULL && existing_vars->start <= vars->start) { + while(existing_vars->next != NULL && existing_vars->start < vars->start) { + debugprintf("Looping to find existing vars ...\n"); existing_vars = existing_vars->next; } - if(existing_vars->next != NULL || existing_vars == vars) { - debugprintf("It seems that I already record this vars %d\n",vars->start); + if(existing_vars->next != NULL || existing_vars->start == vars->start) { + debugprintf("It seems that I already record this vars start packet = %d\n",vars->start); } else { debugprintf("Adding a new entry with this start packet = %d\n",vars->start); |