warning: function types not truly compatible in ISO C

**TomTom68** · 27/07/2011, 07h31

Bonjour,

Et encore une !

Voici le prototype de trois fonctions que je trouve dans socket.h :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
 
/* This is the type we use for generic socket address arguments.
 
   With GCC 2.7 and later, the funky union causes redeclarations or
   uses with any of the listed types to be allowed without complaint.
   G++ 2.7 does not support transparent unions so there we want the
   old-style declaration, too.  */
#if defined __cplusplus || !__GNUC_PREREQ (2, 7) || !defined __USE_GNU
# define __SOCKADDR_ARG		struct sockaddr *__restrict
# define __CONST_SOCKADDR_ARG	__const struct sockaddr *
#else
/* Add more `struct sockaddr_AF' types here as necessary.
   These are all the ones I found on NetBSD and Linux.  */
# define __SOCKADDR_ALLTYPES \
  __SOCKADDR_ONETYPE (sockaddr) \
  __SOCKADDR_ONETYPE (sockaddr_at) \
  __SOCKADDR_ONETYPE (sockaddr_ax25) \
  __SOCKADDR_ONETYPE (sockaddr_dl) \
  __SOCKADDR_ONETYPE (sockaddr_eon) \
  __SOCKADDR_ONETYPE (sockaddr_in) \
  __SOCKADDR_ONETYPE (sockaddr_in6) \
  __SOCKADDR_ONETYPE (sockaddr_inarp) \
  __SOCKADDR_ONETYPE (sockaddr_ipx) \
  __SOCKADDR_ONETYPE (sockaddr_iso) \
  __SOCKADDR_ONETYPE (sockaddr_ns) \
  __SOCKADDR_ONETYPE (sockaddr_un) \
  __SOCKADDR_ONETYPE (sockaddr_x25)
 
# define __SOCKADDR_ONETYPE(type) struct type *__restrict __##type##__;
typedef union { __SOCKADDR_ALLTYPES
	      } __SOCKADDR_ARG __attribute__ ((__transparent_union__));
# undef __SOCKADDR_ONETYPE
# define __SOCKADDR_ONETYPE(type) __const struct type *__restrict __##type##__;
typedef union { __SOCKADDR_ALLTYPES
	      } __CONST_SOCKADDR_ARG __attribute__ ((__transparent_union__));
# undef __SOCKADDR_ONETYPE
#endif
 
extern int bind (int __fd, __CONST_SOCKADDR_ARG __addr, socklen_t __len)
     __THROW;
 
/* Open a connection on socket FD to peer at ADDR (which LEN bytes long).
   For connectionless socket types, just set the default address to send to
   and the only address from which to accept transmissions.
   Return 0 on success, -1 for errors.  */
extern int connect (int __fd, __CONST_SOCKADDR_ARG __addr, socklen_t __len)
     __THROW;
 
/* Read N bytes into BUF from socket FD.
   Returns the number read or -1 for errors.  */
extern ssize_t recv (int __fd, void *__buf, size_t __n, int __flags)
     __THROW;

et voici l'utilisation que j'en fais et qui produit le message d'erreur du titre, le même pour chacune des trois fonctions, incompréhensible :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
int bind(int sockfd, const struct sockaddr *my_addr, socklen_t addrlen) {
...
}
 
int connect(int sockfd, const struct sockaddr *serv_addr, socklen_t addrlen) {
...
}
 
ssize_t recv(int s, void *buf, size_t len, int flags) {
...
}

Je ne trouve pas ce qui peut produire cette erreur, tout me semble correct.
Merci pour votre aide.

**matafan** · 27/07/2011, 10h00

Je suppose que le code que tu donnes n'est pas ton code exact, puisque ce ne sont pas des appels de fonction. Donne plutôt ton code exact.

**TomTom68** · 27/07/2011, 10h49

Ben si, c'est mon code qui sera exécuté dans un hook dans une lib chargée via LD_PRELOAD au démarrage du système.

Tout fonctionne bien, mais j'ai ces warnings que je ne sais pas supprimer.

Pour le modèle d'appel, voir le squelette dans le topic suivant :
http://www.developpez.net/forums/d11...nter-and-void/

**souviron34** · 27/07/2011, 11h24

Envoyé par TomTom68

Je ne trouve pas ce qui peut produire cette erreur, tout me semble correct.

Avec quelle version de gcc tournes-tu ?

**Obsidian** · 27/07/2011, 11h32

Tu n'aurais pas oublié d'inclure <sys/types.h> avant <socket.h> ?

Le second utilise des noms de types qui sont définis dans le premier. Posix 2001 les établit d'emblée, mais les autres programmes doivent le faire explicitement.

Envoyé par man page

NOTES

1 POSIX.1-2001 ne requiert pas l'inclusion de <sys/types.h>, et cet en‐tête n'est pas nécessaire sous Linux. Cependant, il doit être inclus sous certaines implémentations historiques (BSD), et les applications portables devraient probablement l'utiliser.

**TomTom68** · 27/07/2011, 20h38

Bonne idée, mais j'ai pourtant bien mis types.h avant socket :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
#include <sys/types.h>
#include <sys/socket.h>

gcc-3.3.4 glibc-2.3.2

EDIT:

Voilà où j'en suis :
J'ai lu çà :
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34885

De là, j'en suis venu à tester ça :
Modif de socket.h

~~#if defined __cplusplus || !__GNUC_PREREQ (2, 7) || !defined __USE_GNU~~

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
# define __SOCKADDR_ARG		struct sockaddr *__restrict
# define __CONST_SOCKADDR_ARG	__const struct sockaddr *

#else
/* Add more `struct sockaddr_AF' types here as necessary.
These are all the ones I found on NetBSD and Linux. */
# define __SOCKADDR_ALLTYPES \
__SOCKADDR_ONETYPE (sockaddr) \
__SOCKADDR_ONETYPE (sockaddr_at) \
__SOCKADDR_ONETYPE (sockaddr_ax25) \
__SOCKADDR_ONETYPE (sockaddr_dl) \
__SOCKADDR_ONETYPE (sockaddr_eon) \
__SOCKADDR_ONETYPE (sockaddr_in) \
__SOCKADDR_ONETYPE (sockaddr_in6) \
__SOCKADDR_ONETYPE (sockaddr_inarp) \
__SOCKADDR_ONETYPE (sockaddr_ipx) \
__SOCKADDR_ONETYPE (sockaddr_iso) \
__SOCKADDR_ONETYPE (sockaddr_ns) \
__SOCKADDR_ONETYPE (sockaddr_un) \
__SOCKADDR_ONETYPE (sockaddr_x25)

# define __SOCKADDR_ONETYPE(type) struct type *__restrict __##type##__;
typedef union { __SOCKADDR_ALLTYPES
} __SOCKADDR_ARG __attribute__ ((__transparent_union__));
# undef __SOCKADDR_ONETYPE
# define __SOCKADDR_ONETYPE(type) __const struct type *__restrict __##type##__;
typedef union { __SOCKADDR_ALLTYPES
} __CONST_SOCKADDR_ARG __attribute__ ((__transparent_union__));
# undef __SOCKADDR_ONETYPE
#endif

Et là, plus de warning !!!

Encore plus profondément dans l'incompréhension, je compile du C pur et comme le work around GCC date de 2008, je devrais savoir faire ma compilation proprement en 2011, sans avoir à bidouiller socket.h.

Un petit coup de main, merci.

**Obsidian** · 27/07/2011, 21h24

Envoyé par TomTom68

gcc-3.3.4 glibc-2.3.2

Sans être complètement obsolète, ça commence à être un peu vieux. À l'heure à laquelle j'écris, on en est à GCC 4.6.1 : http://gcc.gnu.org/

Sinon, ton message d'erreur se produit à quel endroit ? Je pense que c'est principalement l'absence du mot-clé restrict, présent dans le *.h, qui provoque une différence suffisamment subtile pour que le compilo t'avertisse en mode pedantic.

Pour info, la ligne à l'origine de ce message : http://www.open64.net/doc/df/dc0/kgc...ce.html#l00430

**TomTom68** · 27/07/2011, 21h35

Ben c'est vieux, mais c'est la toolchain de compilation de Tomtom !

Si tu me dis que je peux compiler avec plus récent sans perte de compatibilité avec le core, je veux bien, mais je ne suis pas assez avancé dans ce genre de choix pour m'auto-déterminer sur ce point.

Tu verrais un moyen propre de contourner ce warning inutile ?

Pour info, le message se produit sur la déclaration de chaque fonction.

**Obsidian** · 27/07/2011, 21h41

As-tu essayé de rajouter restrict ?
Peut-on voir également les options que tu passes à GCC ?

**TomTom68** · 27/07/2011, 21h52

Pas sur de savoir faire ce qu'il faut.
J'ai tenté ça :

(Idiot, mais quand on maitrise pas ...)
#define __restrict
ou
#define restrict

Puis
struct sockaddr *__restrict;
#include <sys/types.h>
#include <sys/socket.h>
ou
#include <sys/types.h>
struct sockaddr *__restrict;
#include <sys/socket.h>

Ca marche pas.
Cerveau pas trop présent ce soir.
Peux-tu me guider STP ? Merci

EDIT:
Extrait du makefile :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
CC		= /usr/local/cross/gcc-3.3.4_glibc-2.3.2/arm-linux/bin/gcc
CFLAGS	= -std=c99 -W -Wall -fPIC -pedantic -Os
LDFLAGS	= -ldl -shared

**Melem** · 27/07/2011, 22h33

Mais non ! restrict est un mot-clé du C (99). __restrict sans doute un synonyme de restrict spécifique à gcc.

**TomTom68** · 28/07/2011, 07h31

Wouh !!
Bien fatigué moi, hier soir !!

Bon, on oublie les bêtises que j'ai pu écrire et on reprend.

J'ai fini par faire ça :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
#include <sys/types.h>
#undef __USE_GNU
#include <sys/socket.h> 
#define __USE_GNU

mais à bout d'arguments pour parvenir à forcer les premières formes de déclarations dans socket.h:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
# define __SOCKADDR_ARG		struct sockaddr *__restrict
# define __CONST_SOCKADDR_ARG	__const struct sockaddr *

Est-ce acceptable ou trop moche et dois-je faire autre chose de plus propre ?

J'ai lu ça, mais ça ne m'a pas aidé, si ça peut vous aider à m'orienter :
http://www.lysator.liu.se/c/restrict.html
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34885
http://old.nabble.com/restrict-keywo...d26596271.html

Merci pour votre aide.

**Obsidian** · 28/07/2011, 08h55

Est-ce que ça change quelque chose si tu déclares tes fonctions de cette façon (en oubliant les #define et #undef)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
int bind(int sockfd, const struct sockaddr * restrict my_addr, socklen_t addrlen) {
...
}

**souviron34** · 28/07/2011, 09h28

Envoyé par TomTom68

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
#include <sys/types.h>
#undef __USE_GNU
#include <sys/socket.h> 
#define __USE_GNU

Ce serait plus correct de faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
#include <sys/types.h>
#ifdef __USE_GNU 
#undef __USE_GNU
#define MY_WORKAROUND
#endif
#include <sys/socket.h> 
#ifdef MY_WORKAROUND
#define __USE_GNU
#endif

**TomTom68** · 28/07/2011, 10h15

Envoyé par Obsidian

Citation:
Est-ce que ça change quelque chose si tu déclares tes fonctions de cette façon (en oubliant les #define et #undef)

C'est ce que j'avais fait avant d'en venir aux #define et #undef.

J'avais testé :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
int bind(int sockfd, const struct sockaddr * restrict my_addr, socklen_t addrlen) {
...
int bind(int sockfd, const struct sockaddr * __restrict my_addr, socklen_t addrlen) {
...
int bind(int sockfd, const struct sockaddr * __restrict__ my_addr, socklen_t addrlen) {
...
int bind(int sockfd, __CONST_SOCKADDR_ARG *my_addr, socklen_t addrlen) {
...
int bind(int sockfd, __CONST_SOCKADDR_ARG my_addr, socklen_t addrlen) {
...

sans succès.

**TomTom68** · 28/07/2011, 19h32

Bonsoir,

Donc j'ai reste au workaround ou une idée plus élégante est-elle proposable ?!

Merci pour votre aide.

warning: function types not truly compatible in ISO C

C

Discussions similaires

Partager

Partager