On 18/11/2008, Mladen Turk <[EMAIL PROTECTED]> wrote:
> sebb wrote:
>
> > I happened to spot the following in ssl.c:
> >
> > static int jbs_gets(BIO *b, char *out, int outl)
> > ...
> > int l = (int)strlen(J2S(o));
> > if (l < outl) {
> >    strcpy(out, J2S(o));
> >    ret = outl;
> > }
> > ...
> > return ret;
> >
> > It looks a bit odd to me - I would have expected it to return l rather
> > than outl.
> >
> > However, I've no idea what the function API is supposed to be - and
> > the caller can use strlen(out) to find l -  so I'm probably wrong.
> >
> >
> >
>
>  You are probably right, but the entire BIO code
>  isn't used anywhere. It was something we
>  (Jean-Frederic I think) experimented at the
>  beginning.

Perhaps it should be removed then, or at least commented out.
Seems rather risky to me to have unused JNI code in a release.

>  Regards
>  --
>  ^(TM)
>
> ---------------------------------------------------------------------
>  To unsubscribe, e-mail: [EMAIL PROTECTED]
>  For additional commands, e-mail: [EMAIL PROTECTED]
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to