d3d12: Initialize local_resource member mapped in constructor.
authorVinson Lee <vlee@freedesktop.org>
Tue, 22 Dec 2020 01:52:04 +0000 (17:52 -0800)
committerVinson Lee <vlee@freedesktop.org>
Wed, 30 Dec 2020 05:44:12 +0000 (21:44 -0800)
Fix defect reported by Coverity Scan.

Uninitialized scalar field (UNINIT_CTOR)
uninit_member: Non-static class member mapped is not initialized in this
constructor nor in any functions that it calls.

Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8193>

src/gallium/drivers/d3d12/d3d12_resource.cpp

index 660328a..5ef1b01 100644 (file)
@@ -622,7 +622,8 @@ synchronize(struct d3d12_context *ctx,
 /* A wrapper to make sure local resources are freed and unmapped with
  * any exit path */
 struct local_resource {
-   local_resource(pipe_screen *s, struct pipe_resource *tmpl)
+   local_resource(pipe_screen *s, struct pipe_resource *tmpl) :
+      mapped(false)
    {
       res = d3d12_resource(d3d12_resource_create(s, tmpl));
    }