Re: wrong magic number (using old sources)

[selinux770@tortenboxer.de]

> Ok, so libsepol needs to be fixed in the same way the kernel was to
> eliminate unaligned accesses.  Might take a little bit to work through
> all the code.  The size of the buffer has to be large enough for the
> largest next_entry() request.
> 
> As a possible short term work around, you might try this patch instead.
> It just converts the static buffer used within libsepol from unsigned
> char to uint32_t so that the buffer will be aligned (at least for u32).
> That avoids changing all the callers.

This smal patch worked fine for me. The policy is created and loaded (correctly I think :-)).

Thanks a lot for you support.


 
> Index: libsepol/src/private.h
> ===================================================================
> --- libsepol/src/private.h	(revision 2517)
> +++ libsepol/src/private.h	(working copy)
> @@ -5,6 +5,7 @@
>  #include <byteswap.h>
>  #include <endian.h>
>  #include <sepol/policydb/policydb.h>
> +#include <stdint.h>
>  
>  #if __BYTE_ORDER == __LITTLE_ENDIAN
>  #define cpu_to_le16(x) (x)
> @@ -39,7 +40,7 @@
>  /* Reading from a policy "file". */
>  static inline void *next_entry(struct policy_file *fp, size_t bytes)
>  {
> -	static unsigned char buffer[BUFSIZ];
> +	static uint32_t buffer[20];
>  	size_t nread;
>  
>  	if (bytes > sizeof buffer)
> 
> -- 
> Stephen Smalley
> National Security Agency

-- 
GMX FreeMail: 1 GB Postfach, 5 E-Mail-Adressen, 10 Free SMS.
Alle Infos und kostenlose Anmeldung: http://www.gmx.net/de/go/freemail

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.