[PATCHv2] cifs-utils: work around missing krb5_free_string in Heimdal

Pavel Shilovsky piastryyy at gmail.com
Thu Mar 31 23:12:36 UTC 2022


Thanks for the patch! Merged.

--
Best regards,
Pavel Shilovsky

пт, 4 мар. 2022 г. в 14:58, Jeremy Allison via samba-technical
<samba-technical at lists.samba.org>:
>
> On Sat, Mar 05, 2022 at 12:51:43AM +0200, atheik wrote:
> >On 3/4/22, Jeremy Allison <jra at samba.org> wrote:
> >> Does cifs-utils have a libreplace equivalent ?
> >
> >I have no idea about libreplace but cifs.upcall.c of
> >cifs-utils already has a lot of these #ifdefs for
> >compatibility between MIT and Heimdal KRB5.
> >
> >> Wouldn't it be easier to test for the existence of
> >> krb5_free_string() and write a libreplace wrapper
> >> function so krb5_free_string() can be called from
> >> all code without #ifdefs in the main code ?
> >
> >This patch adds just another #ifdef to cifs.upcall.c
> >so that cifs-utils builds against Heimdal again.
> >
> >I have attached a new patch that doesn't place the
> >#ifdefs in the function body.
>
> That looks cleaner to me, thanks !
>



More information about the samba-technical mailing list