On 2020/04/16 08:42, Giovanni Bechis wrote: > > > @@ -0,0 +1,84 @@ > > > +$OpenBSD$ > > > + > > > +# Memory leak in mod_ssl > > > (https://bz.apache.org/bugzilla/show_bug.cgi?id=63687) > > > > icing's version was already committed upstream, it's probably better to use > > that I think? > > > > https://svn.apache.org/viewvc/httpd/httpd/trunk/modules/ssl/ssl_util_stapling.c?view=log > > > this is icing's version, shouldn't I mention the bz as a referral ?
Ah sorry, I saw your diff at the bottom of the bz and thought it was that - for something committed upstream it's best to include a reference to the commit (I guess https://svn.apache.org/viewvc?view=revision&revision=1876548 is the best reference for that) and maybe the bz as well since it provides more information. OK (could you backport it too, please?)