From dc73064bc51a931d4b9247c36f3f67402643d9dc Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Fri, 5 Nov 2004 12:52:09 +0000 Subject: r3557: Use a switch, not a series of if/else if statements. Andrew Bartlett (This used to be commit fb7bc73f581c8b13041a91a115b4932f76b00ca7) --- source4/torture/rpc/netlogon.c | 19 +++++++++++-------- 1 file changed, 11 insertions(+), 8 deletions(-) (limited to 'source4') diff --git a/source4/torture/rpc/netlogon.c b/source4/torture/rpc/netlogon.c index 6d81b774d8..75f903d8c8 100644 --- a/source4/torture/rpc/netlogon.c +++ b/source4/torture/rpc/netlogon.c @@ -280,6 +280,7 @@ struct samlogon_state { struct netr_LogonSamLogon r; struct netr_Authenticator auth, auth2; struct creds_CredentialState creds; + DATA_BLOB chall; }; @@ -374,14 +375,16 @@ static NTSTATUS check_samlogon(struct samlogon_state *samlogon_state, } /* find and decyrpt the session keys, return in parameters above */ - if (r->in.validation_level == 2) { - base = &r->out.validation.sam2->base; - } else if (r->in.validation_level == 3) { - base = &r->out.validation.sam3->base; - } else if (r->in.validation_level == 6) { - base = &r->out.validation.sam6->base; - } else { - base = NULL; + switch (r->in.validation_level) { + case 2: + base = &r->out.validation.sam2->base; + break; + case 3: + base = &r->out.validation.sam3->base; + break; + case 6: + base = &r->out.validation.sam6->base; + break; } if (r->in.validation_level != 6) { -- cgit