Fix an issue identified by klockwork. (#478)

Diptorup Deb · web-flow · commit 48794f782063 · 2021-05-26T20:03:05.000-05:00
diff --git a/dpctl-capi/source/dpctl_sycl_platform_interface.cpp b/dpctl-capi/source/dpctl_sycl_platform_interface.cpp
@@ -84,11 +84,12 @@ __dpctl_give DPCTLSyclPlatformRef DPCTLPlatform_CreateFromSelector(
             P = new platform(*DS);
             PRef = wrap(P);
         } catch (std::bad_alloc const &ba) {
-            delete P;
             std::cerr << ba.what() << '\n';
+            return nullptr;
         } catch (runtime_error const &re) {
             delete P;
             std::cerr << re.what() << '\n';
+            return nullptr;
         }
     }
     else {

Original file line number	Diff line number	Diff line change
`@@ -84,11 +84,12 @@ __dpctl_give DPCTLSyclPlatformRef DPCTLPlatform_CreateFromSelector(`
`84`	`84`	`P = new platform(*DS);`
`85`	`85`	`PRef = wrap(P);`
`86`	`86`	`} catch (std::bad_alloc const &ba) {`
`87`		`- delete P;`
`88`	`87`	`std::cerr << ba.what() << '\n';`
	`88`	`+ return nullptr;`
`89`	`89`	`} catch (runtime_error const &re) {`
`90`	`90`	`delete P;`
`91`	`91`	`std::cerr << re.what() << '\n';`
	`92`	`+ return nullptr;`
`92`	`93`	`}`
`93`	`94`	`}`
`94`	`95`	`else {`