<div dir="ltr">Hi, thanks for the feedback, I'll apply the changes suggested.<div><br></div><div>> And I just realized after submitting that all errors are ignored<br>> during all the steps... I think the logic I suggested still applies,<br>> though, to keep clear who owns the 'retries' object.<br>> <br>> But now, shouldn't we fail with an error if e.g. none of the retries<br>> checks succeeds?<br></div><div><br></div><div>My idea here is to avoid skipping one or more checks of the retries only because the previous one failed, this is why I used only logging messages to signal the failures (and this is why I used mm_err, in place of mm_warn), but yes, if none of the checks succeeds, return with a failure would be the right thing to do.</div><div><br></div><div>I would put a flag in the Context struct to take trace that at least one check succeeded and at the last step (PUK2) I would check this flag and decide whether return with failure or not. What do you think about that?</div><div><br></div><div>Carlo</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 15 Dec 2015 at 17:41 Aleksander Morgado <<a href="mailto:aleksander@aleksander.es">aleksander@aleksander.es</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Dec 15, 2015 at 5:38 PM, Aleksander Morgado<br>
<<a href="mailto:aleksander@aleksander.es" target="_blank">aleksander@aleksander.es</a>> wrote:<br>
><br>
> Looks like the 'retries' object is not being freed properly in the<br>
> case of errors. What you need to do is to make sure that the ownership<br>
> of the retries object is always transferred properly between the<br>
> different logic parts.<br>
<br>
And I just realized after submitting that all errors are ignored<br>
during all the steps... I think the logic I suggested still applies,<br>
though, to keep clear who owns the 'retries' object.<br>
<br>
But now, shouldn't we fail with an error if e.g. none of the retries<br>
checks succeeds?<br>
<br>
--<br>
Aleksander<br>
<a href="https://aleksander.es" rel="noreferrer" target="_blank">https://aleksander.es</a><br>
</blockquote></div>