Mesa (master): clover: Add constructor for clover::module.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Sat Jan 9 02:38:59 UTC 2021


Module: Mesa
Branch: master
Commit: 5c59e4efe2d5c16cef47ee452c2a189b19a68d19
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=5c59e4efe2d5c16cef47ee452c2a189b19a68d19

Author: Vinson Lee <vlee at freedesktop.org>
Date:   Mon Jan  4 20:27:42 2021 -0800

clover: Add constructor for clover::module.

Fix defect reported by Coverity Scan after commit 95527fe2292
("clover/module: add a printf support to module (v5)").

Uninitialized scalar variable (UNINIT)
uninit_use_in_call: Using uninitialized value
m.printf_strings_in_buffer when calling module.

Signed-off-by: Vinson Lee <vlee at freedesktop.org>
Reviewed-by: Francisco Jerez <currojerez at riseup.net>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8322>

---

 src/gallium/frontends/clover/core/module.hpp | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/gallium/frontends/clover/core/module.hpp b/src/gallium/frontends/clover/core/module.hpp
index 1d1f803520f..b014d3c40cb 100644
--- a/src/gallium/frontends/clover/core/module.hpp
+++ b/src/gallium/frontends/clover/core/module.hpp
@@ -153,6 +153,7 @@ namespace clover {
          std::vector<argument> args;
       };
 
+      module() : printf_strings_in_buffer(0) { }
       void serialize(std::ostream &os) const;
       static module deserialize(std::istream &is);
       size_t size() const;



More information about the mesa-commit mailing list