Merge pull request #2343 from nrspruit/coverity_fixes_l0

[L0] fix Coverity issues in memory and timestamps
oneapi-src · Nov 21, 2024 · 28ef4c8 · 28ef4c8
2 parents c3bcabf + 257225f
commit 28ef4c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/source/adapters/level_zero/event.cpp b/source/adapters/level_zero/event.cpp
@@ -549,7 +549,8 @@ ur_result_t urEventGetProfilingInfo(
 
   // For timestamped events we have the timestamps ready directly on the event
   // handle, so we short-circuit the return.
-  if (isTimestampedEvent) {
+  // We don't support user events with timestamps due to requiring the UrQueue.
+  if (isTimestampedEvent && Event->UrQueue) {
     uint64_t ContextStartTime = Event->RecordEventStartTimestamp;
     switch (PropName) {
     case UR_PROFILING_INFO_COMMAND_QUEUED:

diff --git a/source/adapters/level_zero/memory.cpp b/source/adapters/level_zero/memory.cpp
@@ -2240,7 +2240,7 @@ ur_result_t _ur_buffer::getZeHandle(char *&ZeHandle, access_mode_t AccessMode,
                     nullptr, 0u, nullptr));
       }
       if (waitlist.ZeEventList) {
-        delete waitlist.ZeEventList;
+        delete[] waitlist.ZeEventList;
       }
     }
     Allocation.Valid = true;