diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2018-08-26 17:35:32 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-08-26 17:35:32 +0200 |
| commit | 282c0483c9688de79e0c147eb26638d48fb46ed8 (patch) | |
| tree | 620f92952eff6b7936d94a598aa117ed27faf795 /platform/android | |
| parent | 653b4829f1bee9e6bfae9d353b6e1145cea0aece (diff) | |
| parent | e4af39cbc00446b03d142882a69813e94f0838b2 (diff) | |
Merge pull request #21411 from Crazy-P/fixes-several-resource-leaks
Fixes several resource leaks listed on coverity
Diffstat (limited to 'platform/android')
| -rw-r--r-- | platform/android/export/export.cpp | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/platform/android/export/export.cpp b/platform/android/export/export.cpp index b76b0d5dbe..5c8d9e078f 100644 --- a/platform/android/export/export.cpp +++ b/platform/android/export/export.cpp @@ -1591,8 +1591,11 @@ public: String apkfname = "main." + itos(version_code) + "." + get_package_name(package_name) + ".obb"; String fullpath = p_path.get_base_dir().plus_file(apkfname); err = save_pack(p_preset, fullpath); + if (err != OK) { + unzClose(pkg); EditorNode::add_io_error("Could not write expansion package file: " + apkfname); + return OK; } |