[exim-dev] [Bug 870] Compile warnings for new DKIM code

Top Page
Delete this message
Reply to this message
Author: David Woodhouse
Date:  
To: exim-dev
Subject: [exim-dev] [Bug 870] Compile warnings for new DKIM code
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=870

David Woodhouse <dwmw2@???> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dwmw2@???
           Priority|medium                      |high





--- Comment #1 from David Woodhouse <dwmw2@???> 2009-08-27 12:02:09 ---
The one in expand.c is fairly harmless, and could be handled by just changing
the datatype to a 'long'.

The ones in dkim.c are very much _not_ harmless, AFAICT. When you ask it to
expand $dkim_algo, $dkim_canon_body or $dkim_canon_headers, it looks like it's
going to return the value of an integer field, when it should be returning a
string. This isn't an "I think I can store pointers in an int" bug -- it's
actually returning _real_ integer values, AFAICT. It looks like it'll crash.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email