[PATCH 01/15] staging: vboxvideo: Cleanup vbox_set_up_input_mapping()
Greg Kroah-Hartman
gregkh at linuxfoundation.org
Fri Sep 28 12:38:35 UTC 2018
On Wed, Sep 26, 2018 at 09:41:52PM +0200, Hans de Goede wrote:
> This cleanups 2 things:
>
> 1) The first time we loop over the crtc-s, to compare framebuffers, fb1 may
> get set to NULL by the fb1 = CRTC_FB(crtci); statement and then we call
> to_vbox_framebuffer() on it. The result of this call is only used for
> an address comparison, so we don't end up dereferencing the bad pointer,
> but still it is better to not do this.
>
> 2) Since we already figure out the first crtc with a fb in the first loop
> and store that in fb1, there is no need to loop over the crtc-s again just
> to find the first crtc with a fb again.
>
> Signed-off-by: Hans de Goede <hdegoede at redhat.com>
I have two patch 1/15 in this series, making it 16 patches?
Something went odd on your end, can you please resend these properly?
thanks,
greg k-h
More information about the dri-devel
mailing list