From 582b6333fef858fd7e2a8143517d3cbc6c3670fd Mon Sep 17 00:00:00 2001 From: Daira Hopwood Date: Thu, 22 Oct 2015 14:30:13 +0100 Subject: [PATCH] Add test that we don't write files outside the magic folder directory. refs ticket:2506 Signed-off-by: Daira Hopwood --- src/allmydata/frontends/magic_folder.py | 3 +++ src/allmydata/test/test_magic_folder.py | 34 +++++++++++++++++++++---- src/allmydata/util/deferredutil.py | 2 +- 3 files changed, 33 insertions(+), 6 deletions(-) diff --git a/src/allmydata/frontends/magic_folder.py b/src/allmydata/frontends/magic_folder.py index 995ff2cb..5c4d6c31 100644 --- a/src/allmydata/frontends/magic_folder.py +++ b/src/allmydata/frontends/magic_folder.py @@ -620,6 +620,9 @@ class Downloader(QueueMixin, WriteFileMixin): file_node, metadata = max(self._download_scan_batch[relpath_u], key=lambda x: x[1]['version']) if self._should_download(relpath_u, metadata['version']): extension += [(relpath_u, file_node, metadata)] + else: + self._count('objects_excluded') + self._call_hook(None, 'processed') return extension def _when_queue_is_empty(self): diff --git a/src/allmydata/test/test_magic_folder.py b/src/allmydata/test/test_magic_folder.py index e43a0030..528bb0d5 100644 --- a/src/allmydata/test/test_magic_folder.py +++ b/src/allmydata/test/test_magic_folder.py @@ -20,6 +20,7 @@ from allmydata.frontends import magic_folder from allmydata.frontends.magic_folder import MagicFolder, Downloader, WriteFileMixin from allmydata import magicfolderdb, magicpath from allmydata.util.fileutil import abspath_expanduser_unicode +from allmydata.immutable.upload import Data class MagicFolderTestMixin(MagicFolderCLITestMixin, ShouldFailMixin, ReallyEqualMixin, NonASCIIPathMixin): @@ -347,17 +348,18 @@ class MagicFolderTestMixin(MagicFolderCLITestMixin, ShouldFailMixin, ReallyEqual self.failUnlessReallyEqual(self._get_count('downloader.'+name, client=self.bob_magicfolder._client), expected) + def _wait_for_Bob(ign, downloaded_d): + print "Now waiting for Bob to download\n" + bob_clock.advance(0) + return downloaded_d + def _wait_for(ign, something_to_do): downloaded_d = self.bob_magicfolder.downloader.set_hook('processed') uploaded_d = self.alice_magicfolder.uploader.set_hook('processed') something_to_do() print "Waiting for Alice to upload\n" alice_clock.advance(0) - def _wait_for_Bob(ign): - print "Now waiting for Bob to download\n" - bob_clock.advance(0) - return downloaded_d - uploaded_d.addCallback(_wait_for_Bob) + uploaded_d.addCallback(_wait_for_Bob, downloaded_d) return uploaded_d def Alice_to_write_a_file(): @@ -417,6 +419,28 @@ class MagicFolderTestMixin(MagicFolderCLITestMixin, ShouldFailMixin, ReallyEqual d.addCallback(_check_downloader_count, 'objects_failed', 0) d.addCallback(_check_downloader_count, 'objects_downloaded', 3) + path_u = u"/tmp/magic_folder_test" + encoded_path_u = magicpath.path2magic(u"/tmp/magic_folder_test") + + def Alice_tries_to_p0wn_Bob(ign): + print "Alice tries to p0wn Bob\n" + processed_d = self.bob_magicfolder.downloader.set_hook('processed') + + # upload a file that would provoke the security bug from #2506 + uploadable = Data("", self.alice_magicfolder._client.convergence) + alice_dmd = self.alice_magicfolder.uploader._upload_dirnode + + d2 = alice_dmd.add_file(encoded_path_u, uploadable, metadata={"version": 0}, overwrite=True) + d2.addCallback(lambda ign: self.failUnless(alice_dmd.has_child(encoded_path_u))) + d2.addCallback(_wait_for_Bob, processed_d) + return d2 + d.addCallback(Alice_tries_to_p0wn_Bob) + + d.addCallback(lambda ign: self.failIf(os.path.exists(path_u))) + d.addCallback(lambda ign: self._check_version_in_local_db(self.bob_magicfolder, encoded_path_u, None)) + d.addCallback(_check_downloader_count, 'objects_excluded', 1) + d.addCallback(_check_downloader_count, 'objects_downloaded', 3) + def _cleanup(ign, magicfolder, clock): if magicfolder is not None: d2 = magicfolder.finish() diff --git a/src/allmydata/util/deferredutil.py b/src/allmydata/util/deferredutil.py index 9b37cb27..dac399d7 100644 --- a/src/allmydata/util/deferredutil.py +++ b/src/allmydata/util/deferredutil.py @@ -116,7 +116,7 @@ class HookMixin: """ hook = self._hooks[name] if hook is None: - return defer.succeed(None) + return None (d, ignore_count) = hook self._log("call_hook %r, ignore_count=%r" % (name, ignore_count)) -- 2.37.2