Hi Jan,
On Thu, Aug 16, 2018 at 10:37 AM Emil Velikov <[email protected]> wrote:
>
> Hi Jan,
>
> On 13 July 2018 at 20:57, Jan Palus <[email protected]> wrote:
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107098
> > Signed-off-by: Jan Palus <[email protected]>
> > ---
Thanks for fixing this, actually after digging in some kernel code, it
seems my original assumation that
munmap(NULL...) should be noop is false. So linux kernel will dry to
munmap for address space between
NULL and NULL + len, if you happen to have a "low" mapping which
starts before len, then it will unmap
some process memory.
> Thanks for fixing this issue. For the future, feel free to use a fixes
> tag (+cc) as below.
> It provides a nice reference when picking the fix for stable branches,
> plus the author is your first line reviewer ;-)
>
> Fixes: d891f28df9a ("gallium/winsys/kms: Fix possible leak in map/unmap.")
> Cc: Lepton Wu <[email protected]>
>
> Reviewed-by: Emil Velikov <[email protected]>
>
> Just heading out of the office, so I'll push this tomorrow morning.
>
> -Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
