[apparmor] [patch 22/21] parser: convert to use endian(3) functions (v2)
Steve Beattie
steve at nxnw.org
Tue Mar 18 16:54:02 UTC 2014
[Including this patch in the thread to ensure that it doesn't get
forgotten, but it depends on earlier patches to apply cleanly.]
This patch makes use of the htoleXX() functions (see endian(3))
defined as part of endian.h (already included in parser_interface.c),
instead of defining a function differently based on the detection of
endian related macros.
This fixes a build failure experienced on powerpc with John's patch
set applied. This patch has been updated with John's feedback to use
letoh16() in the le16_to_cpu() macro.
Signed-off-by: Steve Beattie <steve at nxnw.org>
Acked-by: John Johansen <john.johansen at canonical.com>
---
parser/parser.h | 15 ++++++---------
parser/parser_interface.c | 3 ---
parser/parser_main.c | 6 +-----
3 files changed, 7 insertions(+), 17 deletions(-)
Index: b/parser/parser_interface.c
===================================================================
--- a/parser/parser_interface.c
+++ b/parser/parser_interface.c
@@ -37,9 +37,6 @@
#include <unistd.h>
#include <linux/unistd.h>
-/* only for x86 at the moment */
-#include <endian.h>
-#include <byteswap.h>
#include <libintl.h>
#define _(s) gettext(s)
Index: b/parser/parser.h
===================================================================
--- a/parser/parser.h
+++ b/parser/parser.h
@@ -23,9 +23,11 @@
#define __AA_PARSER_H
+#include <endian.h>
#include <string.h>
#include <netinet/in.h>
#include <sys/resource.h>
+
#include "immunix.h"
#include "libapparmor_re/apparmor_re.h"
#include "libapparmor_re/aare_rules.h"
@@ -258,15 +260,10 @@ do { \
#define u16 uint16_t
#define u32 uint32_t
#define u64 uint64_t
-#if __BYTE_ORDER == __BIG_ENDIAN
-# define cpu_to_le16(x) ((u16)(bswap_16 ((u16) x)))
-# define cpu_to_le32(x) ((u32)(bswap_32 ((u32) x)))
-# define cpu_to_le64(x) ((u64)(bswap_64 ((u64) x)))
-#else
-# define cpu_to_le16(x) ((u16)(x))
-# define cpu_to_le32(x) ((u32)(x))
-# define cpu_to_le64(x) ((u64)(x))
-#endif
+
+#define cpu_to_le16(x) ((u16)(htole16 ((u16) x)))
+#define cpu_to_le32(x) ((u32)(htole32 ((u32) x)))
+#define cpu_to_le64(x) ((u64)(htole64 ((u64) x)))
/* The encoding for kernal abi > 5 is
* 28-31: reserved
Index: b/parser/parser_main.c
===================================================================
--- a/parser/parser_main.c
+++ b/parser/parser_main.c
@@ -797,11 +797,7 @@ int test_for_dir_mode(const char *basena
return rc;
}
-#if __BYTE_ORDER == __BIG_ENDIAN
-# define le16_to_cpu(x) ((uint16_t)(bswap_16 (*(uint16_t *) x)))
-#else
-# define le16_to_cpu(x) (*(uint16_t *)(x))
-#endif
+#define le16_to_cpu(x) ((uint16_t)(letoh16 (*(uint16_t *) x)))
const char header_string[] = "\004\010\000version\000\002";
#define HEADER_STRING_SIZE 12
--
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20140318/ebb59b6b/attachment.pgp>
More information about the AppArmor
mailing list