Re: [PATCH] do not free string returned by login_getcapstr

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

 



On Mon, 15 Feb 2021 at 18:58, Damien Miller <djm@xxxxxxxxxxx> wrote:
>
> > NetBSD is idential to OpenBSD. I guess we'll need to special-case FreeBSD
> > and anything else that derives from that codebase. Does anyone know what
> > else does it the FreeBSD way? (I'm guessing Dragonfly...)

I'd assume MidnightBSD as another reply suggested.

> actually, this is in the child process so the leak doesn't matter here.
> I think this fix is fine.

That sounds reasonable to me. Probably with a comment pointing out
that we're intentionally ignoring the leak due to OS differences as a
hint to folks looking at static analysis results in the future. (While
looking at this issue on FreeBSD I saw that Coverity has a false
positive flagging it as a leak.)
_______________________________________________
openssh-unix-dev mailing list
openssh-unix-dev@xxxxxxxxxxx
https://lists.mindrot.org/mailman/listinfo/openssh-unix-dev



[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux