Re: [PATCH v8 0/1] MR8116: kerberos: Add support for SECBUFFER_STREAM to SpUnsealMessage().

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/kerberos/unixlib.c:
>          }
>
>          memcpy( params->data, iov[1].buffer.value, iov[1].buffer.length );
> -
> -
>        /* check whether DATA buffer points within STREAM buffer */
> -
>        if (!((char *)iov[1].buffer.value >= (char *)iov[0].buffer.value && (char *)iov[1].buffer.value < (char *)iov[0].buffer.value + iov[0].buffer.length))
> -
>            pgss_release_buffer( &minor_status, &iov[1].buffer );
> -
>        /* DATA buffer is supposed to point within STREAM buffer, so
> -
>         * there's no need to release it.
> -
>         */

This comment seems unnecessary. Releasing buffers is only needed when GSS_C_BUFFER_FLAG_ALLOCATE is passed.