[OE-core] [PATCH 1/1] cracklib: do_compile failed on RHEL5.8 in which glibc version less than 2.9
Mark Hatle
mark.hatle at windriver.com
Fri May 24 14:50:35 UTC 2013
On 5/24/13 4:56 AM, Hongxu Jia wrote:
> cracklib invokes C functions `be16toh/be32toh/be64toh/htobe16/htobe32/htobe64'
> to fix endian issue on multi platform, but these functions are nonstandard
> which were added to glibc in version 2.9.
>
> The do_compile failed while host's glibc version < 2.9, so use standard
> `htons/htonl/ntohs/ntohl' instead. For the `be64toh/htobe64', there is not
> similar functions, we define `ntohll/htonll' on local to replace.
>
> [YOCTO #4553]
>
> Signed-off-by: Hongxu Jia <hongxu.jia at windriver.com>
I have an alternative fix for this which I finished testing yesterday. I'll
send it up as well and we can determine which is a better fix.
--Mark
> ---
> .../0003-cracklib-fix-do_compile-failed.patch | 189 ++++++++++++++++++++
> meta/recipes-extended/cracklib/cracklib_2.8.22.bb | 4 +-
> 2 files changed, 192 insertions(+), 1 deletion(-)
> create mode 100644 meta/recipes-extended/cracklib/cracklib/0003-cracklib-fix-do_compile-failed.patch
>
> diff --git a/meta/recipes-extended/cracklib/cracklib/0003-cracklib-fix-do_compile-failed.patch b/meta/recipes-extended/cracklib/cracklib/0003-cracklib-fix-do_compile-failed.patch
> new file mode 100644
> index 0000000..d0ef585
> --- /dev/null
> +++ b/meta/recipes-extended/cracklib/cracklib/0003-cracklib-fix-do_compile-failed.patch
> @@ -0,0 +1,189 @@
> +cracklib: fix do_compile failed
> +
> +cracklib invokes C functions `be16toh/be32toh/be64toh/htobe16/htobe32/htobe64'
> +to fix endian issue on multi platform, but these functions are nonstandard
> +which were added to glibc in version 2.9.
> +
> +The do_compile failed while host's glibc version < 2.9, use standard
> +`htons/htonl/ntohs/ntohl' instead. For the `be64toh/htobe64', there is not
> +similar functions, we define `ntohll/htonll' on local to replace.
> +
> +Signed-off-by: Hongxu Jia <hongxu.jia at windriver.com>
> +Upstream-Status: Pending
> +---
> + lib/packlib.c | 75 ++++++++++++++++++++++++++++++++++++++-------------------
> + 1 file changed, 50 insertions(+), 25 deletions(-)
> +
> +diff --git a/lib/packlib.c b/lib/packlib.c
> +index 4b8ccb8..18c21c2 100644
> +--- a/lib/packlib.c
> ++++ b/lib/packlib.c
> +@@ -17,8 +17,7 @@
> + #include <stdint.h>
> + #endif
> +
> +-#define _BSD_SOURCE /* See feature_test_macros(7) */
> +-#include <endian.h>
> ++#include <arpa/inet.h>
> + #include "packer.h"
> +
> + static const char vers_id[] = "packlib.c : v2.3p2 Alec Muffett 18 May 1993";
> +@@ -53,6 +52,32 @@ enum{
> + en_is64
> + };
> +
> ++static uint64_t
> ++ntohll(uint64_t val)
> ++{
> ++ if (__BYTE_ORDER == __LITTLE_ENDIAN)
> ++ {
> ++ return (((uint64_t)htonl((uint32_t)((val << 32) >> 32))) << 32) | (uint32_t)htonl((uint32_t)(val >> 32));
> ++ }
> ++ else if (__BYTE_ORDER == __BIG_ENDIAN)
> ++ {
> ++ return val;
> ++ }
> ++}
> ++
> ++static uint64_t
> ++htonll(uint64_t val)
> ++{
> ++ if (__BYTE_ORDER == __LITTLE_ENDIAN)
> ++ {
> ++ return (((uint64_t)htonl((uint32_t)((val << 32) >> 32))) << 32) | (uint32_t)htonl((uint32_t)(val >> 32));
> ++ }
> ++ else if (__BYTE_ORDER == __BIG_ENDIAN)
> ++ {
> ++ return val;
> ++ }
> ++}
> ++
> + static int
> + IheaderHostToBigEndian(char *pHeader, int nBitType)
> + {
> +@@ -60,10 +85,10 @@ IheaderHostToBigEndian(char *pHeader, int nBitType)
> + {
> + struct pi_header64 *pHeader64 = (struct pi_header64*)pHeader;
> +
> +- pHeader64->pih_magic = htobe64(pHeader64->pih_magic);
> +- pHeader64->pih_numwords = htobe64(pHeader64->pih_numwords);
> +- pHeader64->pih_blocklen = htobe16(pHeader64->pih_blocklen);
> +- pHeader64->pih_pad = htobe16(pHeader64->pih_pad);
> ++ pHeader64->pih_magic = htonll(pHeader64->pih_magic);
> ++ pHeader64->pih_numwords = htonll(pHeader64->pih_numwords);
> ++ pHeader64->pih_blocklen = htons(pHeader64->pih_blocklen);
> ++ pHeader64->pih_pad = htons(pHeader64->pih_pad);
> +
> + #if DEBUG
> + printf("Header64: magic %x, numwords %x, blocklen %x, pad %x\n",
> +@@ -75,10 +100,10 @@ IheaderHostToBigEndian(char *pHeader, int nBitType)
> + {
> + struct pi_header *pHeader32 = (struct pi_header*)pHeader;
> +
> +- pHeader32->pih_magic = htobe32(pHeader32->pih_magic);
> +- pHeader32->pih_numwords = htobe32(pHeader32->pih_numwords);
> +- pHeader32->pih_blocklen = htobe16(pHeader32->pih_blocklen);
> +- pHeader32->pih_pad = htobe16(pHeader32->pih_pad);
> ++ pHeader32->pih_magic = htonl(pHeader32->pih_magic);
> ++ pHeader32->pih_numwords = htonl(pHeader32->pih_numwords);
> ++ pHeader32->pih_blocklen = htons(pHeader32->pih_blocklen);
> ++ pHeader32->pih_pad = htons(pHeader32->pih_pad);
> +
> + #if DEBUG
> + printf("Header32: magic %x, numwords %x, blocklen %x, pad %x\n",
> +@@ -102,10 +127,10 @@ IheaderBigEndianToHost(char *pHeader, int nBitType)
> + {
> + struct pi_header64 *pHeader64 = (struct pi_header64*)pHeader;
> +
> +- pHeader64->pih_magic = be64toh(pHeader64->pih_magic);
> +- pHeader64->pih_numwords = be64toh(pHeader64->pih_numwords);
> +- pHeader64->pih_blocklen = be16toh(pHeader64->pih_blocklen);
> +- pHeader64->pih_pad = be16toh(pHeader64->pih_pad);
> ++ pHeader64->pih_magic = ntohll(pHeader64->pih_magic);
> ++ pHeader64->pih_numwords = ntohll(pHeader64->pih_numwords);
> ++ pHeader64->pih_blocklen = ntohs(pHeader64->pih_blocklen);
> ++ pHeader64->pih_pad = ntohs(pHeader64->pih_pad);
> +
> + #if DEBUG
> + printf("Header64: magic %x, numwords %x, blocklen %x, pad %x\n",
> +@@ -117,10 +142,10 @@ IheaderBigEndianToHost(char *pHeader, int nBitType)
> + {
> + struct pi_header *pHeader32 = (struct pi_header*)pHeader;
> +
> +- pHeader32->pih_magic = be32toh(pHeader32->pih_magic);
> +- pHeader32->pih_numwords = be32toh(pHeader32->pih_numwords);
> +- pHeader32->pih_blocklen = be16toh(pHeader32->pih_blocklen);
> +- pHeader32->pih_pad = be16toh(pHeader32->pih_pad);
> ++ pHeader32->pih_magic = ntohl(pHeader32->pih_magic);
> ++ pHeader32->pih_numwords = ntohl(pHeader32->pih_numwords);
> ++ pHeader32->pih_blocklen = ntohs(pHeader32->pih_blocklen);
> ++ pHeader32->pih_pad = ntohs(pHeader32->pih_pad);
> +
> + #if DEBUG
> + printf("Header32: magic %x, numwords %x, blocklen %x, pad %x\n",
> +@@ -148,7 +173,7 @@ HwmsHostToBigEndian(char *pHwms, int nLen,int nBitType)
> +
> + for (i = 0; i < nLen / sizeof(uint64_t); i++)
> + {
> +- *pHwms64++ = htobe64(*pHwms64);
> ++ *pHwms64++ = htonll(*pHwms64);
> + }
> +
> + }
> +@@ -158,7 +183,7 @@ HwmsHostToBigEndian(char *pHwms, int nLen,int nBitType)
> +
> + for (i = 0; i < nLen / sizeof(uint32_t); i++)
> + {
> +- *pHwms32++ = htobe32(*pHwms32);
> ++ *pHwms32++ = htonl(*pHwms32);
> + }
> +
> + }
> +@@ -192,7 +217,7 @@ HwmsBigEndianToHost(char *pHwms, int nLen, int nBitType)
> +
> + for (i = 0; i < nLen / sizeof(uint64_t); i++)
> + {
> +- *pHwms64++ = be64toh(*pHwms64);
> ++ *pHwms64++ = ntohll(*pHwms64);
> + }
> +
> + }
> +@@ -202,7 +227,7 @@ HwmsBigEndianToHost(char *pHwms, int nLen, int nBitType)
> +
> + for (i = 0; i < nLen / sizeof(uint32_t); i++)
> + {
> +- *pHwms32++ = be32toh(*pHwms32);
> ++ *pHwms32++ = ntohl(*pHwms32);
> + }
> +
> + }
> +@@ -602,7 +627,7 @@ PutPW(pwp, string)
> +
> + datum = (uint32_t) ftell(pwp->dfp);
> +
> +- uint32_t tmpdatum = htobe32(datum);
> ++ uint32_t tmpdatum = htonl(datum);
> + fwrite((char *) &tmpdatum, sizeof(tmpdatum), 1, pwp->ifp);
> +
> + fputs(pwp->data_put[0], pwp->dfp);
> +@@ -671,7 +696,7 @@ GetPW(pwp, number)
> + perror("(index fread failed)");
> + return ((char *) 0);
> + }
> +- datum64 = be64toh(datum64);
> ++ datum64 = ntohll(datum64);
> + datum = datum64;
> + } else {
> + if (fseek(pwp->ifp, sizeof(struct pi_header) + (thisblock * sizeof(uint32_t)), 0))
> +@@ -685,7 +710,7 @@ GetPW(pwp, number)
> + perror("(index fread failed)");
> + return ((char *) 0);
> + }
> +- datum = be32toh(datum);
> ++ datum = ntohl(datum);
> + }
> +
> + int r = 1;
> +--
> +1.7.10.4
> +
> diff --git a/meta/recipes-extended/cracklib/cracklib_2.8.22.bb b/meta/recipes-extended/cracklib/cracklib_2.8.22.bb
> index ae5abc4..77bbf03 100644
> --- a/meta/recipes-extended/cracklib/cracklib_2.8.22.bb
> +++ b/meta/recipes-extended/cracklib/cracklib_2.8.22.bb
> @@ -12,7 +12,9 @@ EXTRA_OECONF = "--without-python"
>
> SRC_URI = "${SOURCEFORGE_MIRROR}/cracklib/cracklib-${PV}.tar.gz \
> file://0001-packlib.c-support-dictionary-byte-order-dependent.patch \
> - file://0002-craklib-fix-testnum-and-teststr-failed.patch"
> + file://0002-craklib-fix-testnum-and-teststr-failed.patch \
> + file://0003-cracklib-fix-do_compile-failed.patch \
> +"
>
> SRC_URI[md5sum] = "463177b5c29c7a598c991e12a4898e06"
> SRC_URI[sha256sum] = "feaff49bfb513ec10b2618c00d2f7f60776ba93fcc5fa22dd3479dd9cad9f770"
>
More information about the Openembedded-core
mailing list