commit 3b35acc913a3b11c7ecf2e3c746fe3d6223c291b
author Bailey Berro <baileyberro@chromium.org> Mon Jul 02 13:58:33 2018 -0700
committer chrome-bot <chrome-bot@chromium.org> Tue Jul 10 09:45:55 2018 -0700
tree f24f1fabf70d6896bc365aaf11d1b615793035a0
parent b124ef6e0b67e6418b723a53e7b24b4dc8d35d4a

smbprovider: Make dtor of DepthFirstIterator virtual

This change makes the destructor of DepthFirstIterator
virtual so that derived classes are properly destroyed.

BUG=chromium:757625
TEST=none
Change-Id: Ib12989d63cf610e8ee373366de13ccf782c59846
Reviewed-on: https://chromium-review.googlesource.com/1123256
Commit-Ready: Bailey Berro <baileyberro@chromium.org>
Tested-by: Bailey Berro <baileyberro@chromium.org>
Reviewed-by: Zentaro Kavanagh <zentaro@chromium.org>

smbprovider/iterator/depth_first_iterator.h

diff --git a/smbprovider/iterator/depth_first_iterator.h b/smbprovider/iterator/depth_first_iterator.h
index d3a1f10..189c1bb 100644
--- a/smbprovider/iterator/depth_first_iterator.h
+++ b/smbprovider/iterator/depth_first_iterator.h
@@ -40,7 +40,7 @@
   DepthFirstIterator(const std::string& dir_path,
                      SambaInterface* samba_interface);
 
-  ~DepthFirstIterator() = default;
+  virtual ~DepthFirstIterator() = default;
 
   // Either OnPop() or OnPush() should be overridden to set |current_| to
   // |entry| in order to achieve a Postorder or Preorder traversal. Without an