aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorIan Jackson <ian.jackson@eu.citrix.com>2013-06-14 16:39:35 +0100
committerIan Jackson <Ian.Jackson@eu.citrix.com>2013-06-14 16:39:35 +0100
commitf7aa72ec00aec71eed055dac5e8a151966d75c9c (patch)
tree2c3361bbbd868dd00fd0ec61dc01a698e2fb316c
parent13e2c808f7ea721c8f200062e2b9b977ee924471 (diff)
downloadxen-f7aa72ec00aec71eed055dac5e8a151966d75c9c.tar.gz
xen-f7aa72ec00aec71eed055dac5e8a151966d75c9c.tar.bz2
xen-f7aa72ec00aec71eed055dac5e8a151966d75c9c.zip
libelf: move include of <asm/guest_access.h> to top of file
libelf-loader.c #includes <asm/guest_access.h>, when being compiled for Xen. Currently it does this in the middle of the file. Move this #include to the top of the file, before libelf-private.h. This is necessary because in forthcoming patches we will introduce private #defines of memcpy etc. which would interfere with definitions in headers #included from guest_access.h. No semantic or functional change in this patch. This is part of the fix to a security issue, XSA-55. Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com> Acked-by: Ian Campbell <ian.campbell@citrix.com> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> Reviewed-by: Chuck Anderson <chuck.anderson@oracle.com>
-rw-r--r--xen/common/libelf/libelf-loader.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/xen/common/libelf/libelf-loader.c b/xen/common/libelf/libelf-loader.c
index bce667ffc7..94257f672e 100644
--- a/xen/common/libelf/libelf-loader.c
+++ b/xen/common/libelf/libelf-loader.c
@@ -16,6 +16,10 @@
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
*/
+#ifdef __XEN__
+#include <asm/guest_access.h>
+#endif
+
#include "libelf-private.h"
/* ------------------------------------------------------------------------ */
@@ -116,7 +120,6 @@ static int elf_load_image(struct elf_binary *elf,
return 0;
}
#else
-#include <asm/guest_access.h>
void elf_set_verbose(struct elf_binary *elf)
{