>> Date: Thu, 18 Aug 2016 21:28:58 +0200 >> >> The kfree() function was called in a few cases by the >> savage_bci_cmdbuf() function during error handling >> even if a passed variable contained a null pointer. >> >> Adjust jump targets according to the Linux coding style convention. >> >> Signed-off-by: Markus Elfring <elfring at users.sourceforge.net> > > Not sure this is worth it, I'll pass. Patch 1 merged.
Unfortunately, it seems that this selection of only one update step from this small patch series has got unwanted consequences. Will the update suggestion â[patch] drm/savage: dereferencing an error pointerâ by Dan Carpenter (from today) trigger further software development discussions? https://patchwork.kernel.org/patch/9372127/ https://lkml.kernel.org/r/<20161012062227.GU12841 at mwanda> Will an update step like â[PATCH 2/2] GPU-DRM-Savage: Less function calls in savage_bci_cmdbuf() after error detectionâ (from 2016-08-18) become worth for related consideratons once more? https://patchwork.kernel.org/patch/9289183/ https://lkml.kernel.org/r/<c97563c0-d463-8b15-5956-26d93641a54f at users.sourceforge.net> Regards, Markus