commit c1bb72c5d299716bf2fe608b671b66b0570a7192
author Corentin Wallez <cwallez@chromium.org> Tue Nov 20 09:14:40 2018 +0000
committer Commit Bot service account <commit-bot@chromium.org> Tue Nov 20 09:14:40 2018 +0000
tree adc5f29add687550aa1ba163b1588b6f3c23a71c
parent fbeca676d409ec19572e3299288f4010c2ee8fd5

DawnWireAndFrontendFuzzer: don't leak the device.

The dawn::Device(dawnDevice) constructor was adding a reference which
caused the device to leak. Fix this by using dawn::Device::Acquire
instead.

BUG=chromium:904734

Change-Id: Id641459f9285c9da51a0d2a1f88126197bfd79a5
Reviewed-on: https://dawn-review.googlesource.com/c/2561
Reviewed-by: Dan Sinclair <dsinclair@google.com>
Reviewed-by: Kai Ninomiya <kainino@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/fuzzers/DawnWireServerAndFrontendFuzzer.cpp

diff --git a/src/fuzzers/DawnWireServerAndFrontendFuzzer.cpp b/src/fuzzers/DawnWireServerAndFrontendFuzzer.cpp
index b08e1d1..7a993df 100644
--- a/src/fuzzers/DawnWireServerAndFrontendFuzzer.cpp
+++ b/src/fuzzers/DawnWireServerAndFrontendFuzzer.cpp
@@ -39,7 +39,7 @@
     dawnProcTable procs = dawn_native::GetProcs();
     dawnSetProcs(&procs);
 
-    dawn::Device nullDevice = dawn_native::null::CreateDevice();
+    dawn::Device nullDevice = dawn::Device::Acquire(dawn_native::null::CreateDevice());
 
     DevNull devNull;
     std::unique_ptr<dawn_wire::CommandHandler> wireServer(