Remove an unnecessary null pointer check

[elfring]

An extra null pointer check is not needed in the function “va_DisplayContextDestroy”.

[elfring]

Is there another update candidate in the function “gl_create_context”?

libva/va/glx/va_glx_impl.c

Lines 435 to 436 in e4dc66b

	if (fbconfigs)
	XFree(fbconfigs);

[XinfengZhang]

please pay attention to "goto Error", the fbconfigs maybe is null.
yes, from the link you provided, free(NULL) is harmless. but when the programmer is not sure about the behavior of Xfree.
it should be ok to double check.
the pre-assumption is: I am not sure whether XFree has null pointer check. but we do here will not have side effect. right?

[elfring]

How do you think about to reduce any remaining uncertainties? 🤔