On Mon, Jun 02, 2014 at 12:27:34PM +0200, Lukas Slebodnik wrote:
On (02/06/14 11:03), Jakub Hrozek wrote:
>Hi,
>
>see attached patch. The bug was preventing SSSD from being built on ARM
>(reproduced on a Raspberri Pi, for example).
>From 86f9acd3579c36fe4b198b8ec63b364620eee6e2 Mon Sep 17 00:00:00 2001
>From: Jakub Hrozek <jhrozek(a)redhat.com>
>Date: Mon, 2 Jun 2014 10:59:46 +0200
>Subject: [PATCH] TESTS: fgetc returns int, not char
>
>We used "char c = fgetc()" where we should have used "int c =
fgetc()"
>This bug caused errors on big-endian architectures.
>---
> src/tests/sss_config-tests.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/src/tests/sss_config-tests.c b/src/tests/sss_config-tests.c
>index
18ce8d8bf8e8abc2afea2350a0d4e8274f4ffc77..1b2608978affbc7cc1a253b9863fd691117757a4 100644
>--- a/src/tests/sss_config-tests.c
>+++ b/src/tests/sss_config-tests.c
>@@ -261,7 +261,7 @@ check_file_content(const char *filename, const char *expected)
> {
> FILE *file = NULL;
> size_t i;
>- char c;
>+ int c;
> bool result;
>
> file = fopen(filename, "r");
>--
>1.9.0
>
Tested with
make sss_config-tests CFLAGS="-Wconversion -Wno-sign-conversion" CC=gcc
CC src/tests/sss_config_tests-sss_config-tests.o
src/tests/sss_config-tests.c: In function ‘check_file_content’:
src/tests/sss_config-tests.c:271:22: warning: conversion to ‘char’ from ‘int’ may alter
its value [-Wconversion]
while ((c = fgetc(file)) != EOF) {
^
CCLD sss_config-tests
The patch fixed warning.
ACK
LS
Pushed to master: b51646c8a3a35a2728f32819a8fdeef822ff1ff2