[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Minios-devel] [UNIKRAFT PATCH] lib/posix-user: Include <stdio.h> within grp.h



Hi Simon, looks good, thanks.

-- Felipe

Reviewed-by: Felipe Huici <felipe.huici@xxxxxxxxx>

On 07.02.20, 10:00, "Simon Kuenzer" <simon.kuenzer@xxxxxxxxx> wrote:

    As soon as _GNU_SOURCE is defined, the header `grp.h` is referring
    to the `FILE` data type. We include `stdio.h` to provide its
    definition.
    
    Signed-off-by: Simon Kuenzer <simon.kuenzer@xxxxxxxxx>
    ---
     lib/posix-user/musl-imported/include/grp.h | 1 +
     1 file changed, 1 insertion(+)
    
    diff --git a/lib/posix-user/musl-imported/include/grp.h 
b/lib/posix-user/musl-imported/include/grp.h
    index a35e61a3..c738eef0 100644
    --- a/lib/posix-user/musl-imported/include/grp.h
    +++ b/lib/posix-user/musl-imported/include/grp.h
    @@ -10,6 +10,7 @@ extern "C" {
     
     #ifdef _GNU_SOURCE
     #define __NEED_FILE
    +#include <stdio.h>
     #endif
     
     #include <sys/types.h>
    -- 
    2.20.1
    
    

_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.