wbrawner/openssl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Use for loop in WPACKET_fill_lengths instead of do...while

Browse source 
Based on review feedback

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/2259)
This commit is contained in:
Matt Caswell 2017-01-27 12:05:52 +00:00
parent 34f7245ba0
commit fed60a781c
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
ssl/packet.c
View file

@ -232,12 +232,10 @@ int WPACKET_fill_lengths(WPACKET *pkt)
if (pkt->subs == NULL) if (pkt->subs == NULL)
return 0; return 0;
sub = pkt->subs; for (sub = pkt->subs; sub != NULL; sub = sub->parent) {
do {
if (!wpacket_intern_close(pkt, sub, 0)) if (!wpacket_intern_close(pkt, sub, 0))
return 0; return 0;
sub = sub->parent; }
} while (sub != NULL);
return 1; return 1;
} }