mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-05-26 23:18:12 +02:00
41642bdbca
If FindProcIndex in egldispatchstubs.c is called with a name that's less than the first entry in the array, it would end up trying to store an index of -1 in an unsigned integer, wrap around to 2^32, and then crash when it tries to look that up. Change FindProcIndex so that it uses bsearch(3) instead of implementing its own binary search, like the GLX equivalent FindGLXFunction does. Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
106 lines
2.9 KiB
C
106 lines
2.9 KiB
C
#include "egldispatchstubs.h"
|
|
#include "g_egldispatchstubs.h"
|
|
|
|
#include <string.h>
|
|
#include <stdlib.h>
|
|
|
|
#include "eglcurrent.h"
|
|
|
|
static const __EGLapiExports *exports;
|
|
|
|
const int __EGL_DISPATCH_FUNC_COUNT = __EGL_DISPATCH_COUNT;
|
|
int __EGL_DISPATCH_FUNC_INDICES[__EGL_DISPATCH_COUNT + 1];
|
|
|
|
static int Compare(const void *l, const void *r)
|
|
{
|
|
const char *s = *(const char **)r;
|
|
return strcmp(l, s);
|
|
}
|
|
|
|
static int FindProcIndex(const char *name)
|
|
{
|
|
const char **match = bsearch(name, __EGL_DISPATCH_FUNC_NAMES,
|
|
__EGL_DISPATCH_COUNT, sizeof(const char *), Compare);
|
|
|
|
if (match == NULL)
|
|
return __EGL_DISPATCH_COUNT;
|
|
|
|
return match - __EGL_DISPATCH_FUNC_NAMES;
|
|
}
|
|
|
|
void __eglInitDispatchStubs(const __EGLapiExports *exportsTable)
|
|
{
|
|
int i;
|
|
exports = exportsTable;
|
|
for (i=0; i<__EGL_DISPATCH_FUNC_COUNT; i++) {
|
|
__EGL_DISPATCH_FUNC_INDICES[i] = -1;
|
|
}
|
|
}
|
|
|
|
void __eglSetDispatchIndex(const char *name, int dispatchIndex)
|
|
{
|
|
int index = FindProcIndex(name);
|
|
__EGL_DISPATCH_FUNC_INDICES[index] = dispatchIndex;
|
|
}
|
|
|
|
void *__eglDispatchFindDispatchFunction(const char *name)
|
|
{
|
|
int index = FindProcIndex(name);
|
|
return (void *) __EGL_DISPATCH_FUNCS[index];
|
|
}
|
|
|
|
static __eglMustCastToProperFunctionPointerType FetchVendorFunc(__EGLvendorInfo *vendor,
|
|
int index, EGLint errorCode)
|
|
{
|
|
__eglMustCastToProperFunctionPointerType func = NULL;
|
|
|
|
if (vendor != NULL) {
|
|
func = exports->fetchDispatchEntry(vendor, __EGL_DISPATCH_FUNC_INDICES[index]);
|
|
}
|
|
if (func == NULL) {
|
|
if (errorCode != EGL_SUCCESS) {
|
|
_eglError(errorCode, __EGL_DISPATCH_FUNC_NAMES[index]);
|
|
}
|
|
return NULL;
|
|
}
|
|
|
|
if (!exports->setLastVendor(vendor)) {
|
|
// Don't bother trying to set an error code in libglvnd. If
|
|
// setLastVendor failed, then setEGLError would also fail.
|
|
_eglError(errorCode, __EGL_DISPATCH_FUNC_NAMES[index]);
|
|
return NULL;
|
|
}
|
|
|
|
return func;
|
|
}
|
|
|
|
__eglMustCastToProperFunctionPointerType __eglDispatchFetchByCurrent(int index)
|
|
{
|
|
__EGLvendorInfo *vendor;
|
|
|
|
// Note: This is only used for the eglWait* functions. For those, if
|
|
// there's no current context, then they're supposed to do nothing but
|
|
// return success.
|
|
exports->threadInit();
|
|
vendor = exports->getCurrentVendor();
|
|
return FetchVendorFunc(vendor, index, EGL_SUCCESS);
|
|
}
|
|
|
|
__eglMustCastToProperFunctionPointerType __eglDispatchFetchByDisplay(EGLDisplay dpy, int index)
|
|
{
|
|
__EGLvendorInfo *vendor;
|
|
|
|
exports->threadInit();
|
|
vendor = exports->getVendorFromDisplay(dpy);
|
|
return FetchVendorFunc(vendor, index, EGL_BAD_DISPLAY);
|
|
}
|
|
|
|
__eglMustCastToProperFunctionPointerType __eglDispatchFetchByDevice(EGLDeviceEXT dev, int index)
|
|
{
|
|
__EGLvendorInfo *vendor;
|
|
|
|
exports->threadInit();
|
|
vendor = exports->getVendorFromDevice(dev);
|
|
return FetchVendorFunc(vendor, index, EGL_BAD_DEVICE_EXT);
|
|
}
|
|
|