Repository: ignite Updated Branches: refs/heads/ignite-2.5 e190e35a9 -> d1ecf732d
IGNITE-8404 Fixed NPE in MappedFileMemoryProvider Project: http://git-wip-us.apache.org/repos/asf/ignite/repo Commit: http://git-wip-us.apache.org/repos/asf/ignite/commit/d1ecf732 Tree: http://git-wip-us.apache.org/repos/asf/ignite/tree/d1ecf732 Diff: http://git-wip-us.apache.org/repos/asf/ignite/diff/d1ecf732 Branch: refs/heads/ignite-2.5 Commit: d1ecf732d1d998afbb526dcf16c2314cebcb82a5 Parents: 4296fdd Author: Alexey Goncharuk <[email protected]> Authored: Thu Apr 26 19:38:03 2018 +0300 Committer: Alexey Goncharuk <[email protected]> Committed: Thu Apr 26 20:02:26 2018 +0300 ---------------------------------------------------------------------- .../mem/file/MappedFileMemoryProvider.java | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ignite/blob/d1ecf732/modules/core/src/main/java/org/apache/ignite/internal/mem/file/MappedFileMemoryProvider.java ---------------------------------------------------------------------- diff --git a/modules/core/src/main/java/org/apache/ignite/internal/mem/file/MappedFileMemoryProvider.java b/modules/core/src/main/java/org/apache/ignite/internal/mem/file/MappedFileMemoryProvider.java index 3800214..7186b27 100644 --- a/modules/core/src/main/java/org/apache/ignite/internal/mem/file/MappedFileMemoryProvider.java +++ b/modules/core/src/main/java/org/apache/ignite/internal/mem/file/MappedFileMemoryProvider.java @@ -94,14 +94,18 @@ public class MappedFileMemoryProvider implements DirectMemoryProvider { /** {@inheritDoc} */ @Override public void shutdown() { - for (MappedFile file : mappedFiles) { - try { - file.close(); - } - catch (IOException e) { - log.error("Failed to close memory-mapped file upon stop (will ignore) [file=" + - file.file() + ", err=" + e.getMessage() + ']'); + if (mappedFiles != null) { + for (MappedFile file : mappedFiles) { + try { + file.close(); + } + catch (IOException e) { + log.error("Failed to close memory-mapped file upon stop (will ignore) [file=" + + file.file() + ", err=" + e.getMessage() + ']'); + } } + + mappedFiles = null; } }
