Add security thoughts.

svn path=/trunk/; revision=4144

diff --git a/design-notes.html b/design-notes.html
index ffd82bf80f746b2ea9b0d6ac41cb7819a1dc4adb..149d67b4800d6ecc0eaedf10dff0527b6d924a8d 100644 (file)
--- a/design-notes.html
+++ b/design-notes.html
@@ -33,6 +33,23 @@ ESR made previously, and the differences and new directions will be laid
 out in this document. It is therefore a sort of a TODO document, until
 the necessary code revisions have been made.</p>
 
+<h2>Security</h2>
+
+<p>Fetchmail was handed over in a pretty poor shape, security-wise. It will
+happily talk to the network with root privileges, use sscanf() to read
+remotely received data into fixed-length stack-based buffers without
+length limitation and so on. A full audit is required and security
+concepts will have to be applied. Random bits are:</p>
+
+<ul>
+    <li>code talking to the network does not require root privileges and
+    needs to run without root permissions</li>
+    <li>all input must be validated, all strings must be length checked,
+    all integers range checked</li>
+    <li>all types will need to be reviewed whether they are signed or
+    unsigned</li>
+</ul>
+
 <h2>SMTP forwarding</h2>
 
 <p>Fetchmails multidrop and rewrite options will process addresses