ULT DriverDllLoader is buggy dealing with symbols and corrupts its own state

[dvrogozh]

media-driver/media_driver/linux/ult/ult_app/driver_loader.h

Line 90 in 7a4a6b4

for (auto i = 0; i < sizeof(this) / sizeof(const void *); i++)

sizeof(this)/sizeof(void*) is obviously equal to 1. The intended use of DriverSymbols::Clear() is eventually to NULL pointers to loaded symbols. Instead _vaDriverInit is NULLed only, i.e. state is corrupted since other pointers were not zeroed.

This has a side effect that memory leak checker still works though DriverSymbols::Clear() is called right before it, i.e. these pointers are still valid.

media-driver/media_driver/linux/ult/ult_app/memory_leak_detector.cpp

Line 37 in 7a4a6b4

int32_t memNinjaCnt = drvLoader.GetDriverSymbols().MOS_GetMemNinjaCounter();