> In qxl_add_mode(), the return value of drm_cvt_mode() is assigned to mode, > which will lead to a possible NULL pointer dereference on failure of > drm_cvt_mode(). Add a check to avoid npd. 1. Can a wording approach (like the following) be a better change description? A null pointer is stored in the local variable “mode” after a call of the function “drm_cvt_mode” failed. This pointer was used in subsequent statements where an undesirable dereference will be performed then. Thus add a corresponding return value check. 2. Would you like to add any tags (like “Fixes” and “Cc”) accordingly? 3. How do you think about to append parentheses to the function name in the summary phrase? Regards, Markus
