<div dir="ltr">2016-06-26 20:24 GMT+02:00 Kurt Roeckx <span dir="ltr"><<a href="mailto:kurt@roeckx.be" target="_blank">kurt@roeckx.be</a>></span>:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On Sun, Jun 26, 2016 at 06:53:42PM +0200, Jérémy Lal wrote:<br>
><br>
> I'm on it, and after a couple things i could solve, i need a "gentle push"<br>
> to continue solving these:<br>
<br>
</span>They all seem to be about the same problem.  The structure has<br>
become opaque and you can't have it directly on the stack or in an<br>
other struct.  Instead you need to allocate it with TYPE_new(),<br>
which will give you a pointer back.  This will ensure that the<br>
allocated size is the correct one, since we might change the<br>
structure to add new fields and thing like that.  When you do<br>
things like sizeof(TYPE) you would get the size at compile time<br>
which might be a different one than the one at runtime.<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>These changes seem quite easy and doable for a novice like me.</div><div>However the whole part about BIO seems to be much harder to fix:</div><div><a href="https://github.com/nodejs/node/blob/v4.x/src/node_crypto_bio.h">https://github.com/nodejs/node/blob/v4.x/src/node_crypto_bio.h</a><br></div><div><a href="https://github.com/nodejs/node/blob/v4.x/src/node_crypto_bio.cc">https://github.com/nodejs/node/blob/v4.x/src/node_crypto_bio.cc</a><br></div><div><br></div><div>Sadly, upstream is not considering moving soon to openssl 1.1.</div><div><a href="https://github.com/nodejs/node/issues/4270">https://github.com/nodejs/node/issues/4270</a><br></div><div><br></div><div>Jérémy.</div><div><br></div></div></div></div>