2018-04-01 15:36:45 +02:00
|
|
|
Implementation notes
|
|
|
|
====================
|
|
|
|
|
2018-04-29 21:41:34 +02:00
|
|
|
Lightweight cleaning mode
|
|
|
|
-------------------------
|
|
|
|
|
|
|
|
Due to *popular* request, MAT2 is providing a *lightweight* cleaning mode,
|
|
|
|
that only cleans the superficial metadata of your file, but not
|
2018-08-30 23:11:35 +02:00
|
|
|
the ones that might be in **embedded** resources. Like for example,
|
2018-04-29 21:41:34 +02:00
|
|
|
images in a PDF or an office document.
|
|
|
|
|
2018-07-01 23:27:24 +02:00
|
|
|
Revisions handling
|
|
|
|
------------------
|
|
|
|
|
2019-05-16 20:59:15 +02:00
|
|
|
Revisions are handled according to the principle of least astonishment: they
|
|
|
|
are entirely removed.
|
2018-07-01 23:27:24 +02:00
|
|
|
|
2019-05-16 20:59:15 +02:00
|
|
|
- Either the users aren't aware of the revisions, are thus they should be
|
|
|
|
deleted. For example journalists that are editing a document to erase
|
|
|
|
mentions sources mentions.
|
2018-07-01 23:27:24 +02:00
|
|
|
|
2019-05-16 20:59:15 +02:00
|
|
|
- Or they are aware of it, and will likely not expect MAT2 to be able to keep
|
|
|
|
the revisions, that are basically traces about how, when and who edited the
|
|
|
|
document.
|
2018-07-01 23:27:24 +02:00
|
|
|
|
|
|
|
|
2018-05-06 21:58:31 +02:00
|
|
|
Race conditions
|
|
|
|
---------------
|
|
|
|
|
|
|
|
MAT2 does its very best to avoid crashing at runtime. This is why it's checking
|
|
|
|
if the file is valid __at parser creation__. MAT2 doesn't take any measure to
|
|
|
|
ensure that the file is not changed between the time the parser is
|
|
|
|
instantiated, and the call to clean or show the metadata.
|
|
|
|
|
2018-04-01 15:36:45 +02:00
|
|
|
Symlink attacks
|
|
|
|
---------------
|
|
|
|
|
|
|
|
MAT2 output predictable filenames (like yourfile.jpg.cleaned).
|
|
|
|
This may lead to symlink attack. Please check if you OS prevent
|
|
|
|
against them
|
|
|
|
|
|
|
|
Archives handling
|
|
|
|
-----------------
|
|
|
|
|
2019-05-16 20:59:15 +02:00
|
|
|
By default, when cleaning a non-support file format in an archive,
|
|
|
|
mat2 will abort with a detailed error message.
|
|
|
|
While strongly discouraged, it's possible to override this behaviour to force
|
|
|
|
the exclusion, or inclusion of unknown files into the cleaned archive.
|
|
|
|
|
|
|
|
While Python's [zipfile](https://docs.python.org/3/library/zipfile.html) module
|
|
|
|
provides *safe* way to extract members of a zip archive, the
|
|
|
|
[tarfile](https://docs.python.org/3/library/tarfile.html) one doesn't,
|
|
|
|
meaning that it's up to mat2 to implement safety checks. Currently,
|
|
|
|
it defends against path-traversal, both relative and absolute,
|
|
|
|
symlink-related attacks, setuid/setgid attacks, duplicate members, block and
|
|
|
|
char devices, … but there might still be dragons lurking there.
|
|
|
|
|
2018-04-01 15:36:45 +02:00
|
|
|
|
|
|
|
PDF handling
|
|
|
|
------------
|
|
|
|
|
|
|
|
MAT was doing some kind of rendering for PDF files, on a cairo surface, then
|
|
|
|
printed it to a file. This kept the text selectable, but unfortunately, it
|
|
|
|
didn't remove any *deep metadata*, like the ones in embedded pictures. This was
|
|
|
|
on of the reason MAT was abandoned: the absence of satisfying solution to
|
|
|
|
handle PDF. But apparently, people are ok with [pdf redact
|
|
|
|
tools](https://github.com/firstlookmedia/pdf-redact-tools), that simply
|
|
|
|
transform the PDF into images. So this is what's MAT2 is doing too.
|
|
|
|
|
2018-04-03 21:45:05 +02:00
|
|
|
Of course, it would be possible to detect images in PDf file, and process them
|
|
|
|
with MAT2, but since a PDF can contain a lot of things, like images, videos,
|
|
|
|
javascript, pdf, blobs, … this is the easiest and safest way to clean them.
|
|
|
|
|
2018-04-01 15:36:45 +02:00
|
|
|
Images handling
|
|
|
|
---------------
|
|
|
|
|
|
|
|
When possible, images are handled like PDF: rendered on a surface, then saved
|
|
|
|
to the filesystem. This ensures that every metadata is removed.
|
|
|
|
|
2018-09-05 18:41:08 +02:00
|
|
|
XML attacks
|
|
|
|
-----------
|
|
|
|
|
2018-09-05 19:00:28 +02:00
|
|
|
Since our threat model conveniently excludes files crafted to specifically
|
2018-09-05 18:41:08 +02:00
|
|
|
bypass MAT2, fileformats containing harmful XML are out of our scope.
|
|
|
|
But since MAT2 is using [etree](https://docs.python.org/3/library/xml.html#xml-vulnerabilities)
|
|
|
|
to process XML, it's "only" vulnerable to DoS, and not memory corruption:
|
|
|
|
odds are that the user will notice that the cleaning didn't succeed.
|