On Thu, 03 Dec 2015, Thomas Hellstrom <thellstrom@xxxxxxxxxx> wrote: > Hi, > > On 12/03/2015 12:12 PM, Jani Nikula wrote: >> On Wed, 02 Dec 2015, Thomas Hellstrom <thellstrom@xxxxxxxxxx> wrote: >>> A client calling drmSetMaster() using a file descriptor that was opened >>> when another client was master would inherit the latter client's master >>> object and all its authenticated clients. >>> >>> This is unwanted behaviour, and when this happens, instead allocate a >>> brand new master object for the client calling drmSetMaster(). >>> >>> Fixes a BUG() throw in vmw_master_set(). >>> >>> Cc: <stable@xxxxxxxxxxxxxxx> >>> Signed-off-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx> >> Drive-by bikeshedding, the actual change might look neater (and easier >> to revert if something falls apart) if you extracted the >> drm_new_set_master() abstraction as a separate non-functional prep >> patch. Not insisting, just a thought. >> >> BR, >> Jani. > While you're probably right, I prefer to have this as a single patch to > avoid ending up in a backporting nightmare for stable. That's reasonable. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html