Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 244480 - mail-mta/ssmtp-2.62-r3: cram-md5 does not work on x64 architecture.
Summary: mail-mta/ssmtp-2.62-r3: cram-md5 does not work on x64 architecture.
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: High normal (vote)
Assignee: Net-Mail Packages
URL: http://bugs.debian.org/cgi-bin/bugrep...
Whiteboard:
Keywords:
Depends on: 304069
Blocks:
  Show dependency tree
 
Reported: 2008-10-26 15:19 UTC by Ingo
Modified: 2011-01-19 13:49 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Fixes UINT5 type definition, so it works for AMD64 platform. (Should also work for 32 bit platforms!) (ssmtp-2.62-x64.patch,490 bytes, patch)
2008-10-26 15:43 UTC, Ingo
Details | Diff
Fixes UINT4 type definition by using u_int32_t instead, (amd64.patch,5.18 KB, patch)
2008-10-26 15:45 UTC, Ingo
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ingo 2008-10-26 15:19:57 UTC
cram-md5 authentication does not work on AMD64 platform.

Reproducible: Always




Here is a super simple [PATCH] which fixes the problem and which should also work on every 32 bit architecture:

------------------------ SNIP -----------------------------------------------
Fix UINT4 type definition for AMD64. Will also work on 32 bit architecture.

--- md5auth/global.h    2002-09-27 14:16:24.000000000 +0200
+++ md5auth/global.h    2008-10-26 16:04:38.000000000 +0100
@@ -17,7 +17,7 @@
 typedef unsigned short int UINT2;

 /* UINT4 defines a four byte word */
-typedef unsigned long int UINT4;
+typedef unsigned int UINT4;

 /* PROTO_LIST is defined depending on how PROTOTYPES is defined above.
 If using PROTOTYPES, then PROTO_LIST returns the list, otherwise it

------------------------ SNIP -----------------------------------------------
Comment 1 Ingo 2008-10-26 15:43:00 UTC
Created attachment 169938 [details, diff]
Fixes UINT5 type definition, so it works for AMD64 platform. (Should also work for 32 bit platforms!)

I think it would be a lot cleaner to use "u_int16_t" and "u_int32_t", but this would require to include "sys/types.h". Not sure if this is wanted.
Comment 2 Ingo 2008-10-26 15:45:48 UTC
Created attachment 169940 [details, diff]
Fixes UINT4 type definition by using u_int32_t instead,

This patch uses "u_int32_t" from "sys/types.h", but unfortunately this means a bigger patch.
Choose which one you like
Comment 3 Tobias Scherbaum (RETIRED) gentoo-dev 2008-11-29 20:23:55 UTC
Did you report those patches upstream? 
Comment 4 Ingo 2009-02-21 15:35:00 UTC
(In reply to comment #3)
> Did you report those patches upstream? 
> 

Sorry for answering so late.
It seems I did not get this via regular E-Mail.

I didn't report this upstream.
But now that you mentioned it I mailed the patch to
397149@bugs.debian.org
and
A.Monsalve.Salazar@IEEE.org  (the Maintainer of ssmtp ?)

I hope this helps ?


Comment 5 Ingo 2009-08-15 11:30:36 UTC
> ...
> But now that you mentioned it I mailed the patch to
> 397149@bugs.debian.org
> and
> A.Monsalve.Salazar@IEEE.org  (the Maintainer of ssmtp ?)
> 
> I hope this helps ?
> 

Currently there seems to be no activity upstream.
Would it be possible to just integrate this patch into gentoo ?

so long
  Ingo

PS: Just updated my system and again had to apply the patch myself...
Comment 6 Mark Gannon 2009-09-18 17:20:05 UTC
I've also encountered this issue and would like to see the patch applied to the official ebuild.
Comment 7 Diego Elio Pettenò (RETIRED) gentoo-dev 2011-01-19 13:49:57 UTC
This was fixed upstream for 2.64 so it's in tree now.