Retour de pointeur

Bonjour,

Dans le cadre du développement d'une extension PHP (écrit en C), je me retrouve dans le cas suivant :

Code:

---

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

const char * phpgit_working_directory_get_branch(git_repository *repo, int format) { int error = 0; const char *branch = NULL; git_reference *head = NULL;   /* Je pense que le code suivant n'est pas déterminant du problème, mais comme je suis un noob en C, je le laisse au cas où */ error = git_repository_head(&head, repo);   if (error == GIT_EUNBORNBRANCH || error == GIT_ENOTFOUND) { zend_throw_exception("Exception", "Branch not found", 404); } else if (!error) { branch = git_reference_shorthand(head); } else { zend_throw_exception("Exception", "Failed to get current branch", 500); }   if (!branch) { if (format == FORMAT_LONG) { zend_throw_exception("Exception", "Not currently on any branch", 0); } else { zend_throw_exception("Exception", "HEAD (no branch)", 1); } }   /* et on retourne branch, qui est un pointeur NULL, ou un pointeur "valide" définit par git_reference_shorthand, qui renvoi "const char *" ( https://libgit2.github.com/libgit2/#v0.23.2/group/reference/git_reference_shorthand ) */   return branch; }

---

Cette fonction, phpgit_working_directory_get_branch, est appelée par une autre fonction

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

PHP_METHOD(WorkingDirectory, getBranch) { php_working_directory_t *wd; git_repository *repo; const char *branch; zend_string zbranch;   wd = PHP_GIT_WORKING_DIRECTORY_FETCH_FROM(Z_OBJ_P(getThis())); repo = wd->repo;   /* La ligne à problème */ branch = phpgit_working_directory_get_branch(repo, FORMAT_SHORT);   if(branch != NULL) { zbranch = zend_string_init(branch, strlen(branch), 0); RETURN_STR(zbranch); } else { RETURN_FALSE(); } }

---

En l'état, je prends un warning :

Citation:

---

warning: assignment makes pointer from integer without a cast

---

Si je cast :

Code:

---

branch = (const char *) phpgit_working_directory_get_branch(repo, FORMAT_SHORT);

---

Je prends un autre warning :

Citation:

---

warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

---

Du coup j'ai deux questions :

1. Pourquoi j'ai besoin de caster un type en un type exactement identique : const char* en const char* ?
2. Mes recherches sur le 2e warning m'amène à penser que phpgit_working_directory_get_branch me retourne en fait un int (typiquement l'adresse vers laquelle pointe ma variable branch, mais du coup je en comprends pas pourquoi) et que il essai de caster cet int en pointeur à nouveau, et que ça lui pose un problème, et la non plus je ne comprends pas pourquoi car cet int d'adresse aurait la même taille puisqu'il vient d'un pointeur, casté en pointeur, sauf que le warning dit que j'ai tord. Du coup je suis perdu, d'où mon message :)

Oula, si le int représente un pointeur, c'est vraiment très moche et très dangereux (si l'adresse du pointeur est en 64 bits alors que ton int est en 32, ça va poser souci). Si tu es absolument sûr que c'est bien un pointeur qui est renvoyé, je crains que tu ne puisses pas faire grand chose d'autre que d'ignorer le warning.

Citation:

---

Oula, si le int représente un pointeur, c'est vraiment très moche et très dangereux

---

Je suis bien d'accord.
Le prototype de la fonction est :

Code:

---

const char * phpgit_working_directory_get_branch(git_repository *repo, int format);

---

Si autre chose qu'un pointeur vers un "const char" était renvoyé, j'aurais une erreur dès la compilation non ?
Ca me ramène au premier warning :

Citation:

---

warning: assignment makes pointer from integer without a cast

---

Il me dit que je reçois un int non ?

Il n'y aurait pas, par hasard, une erreur dans le prototype ou la définition de la fonction?
Est-ce que l'en-tête et le .c définissant la fonction annonce bien const char*?

Ça, ça ressemble au problème typique de manque de warning sur l'usage d'une fonction non-déclarée (je me suis déjà brûlé sur GCC avec ça).

Assure-toi donc que la fonction phpgit_working_directory_get_branch() est bien déclarée correctement là où tu l'utilises.

Pas d'autre message d'erreur avant.

Effectivement je me suis emmêlé avec les h et les protos, j'avais commenté le proto de `phpgit_working_directory_get_branch` dans le h, du coup il y avait déclaration implicite, d'où le warning !
Maintenant j'ai plein de nouvelles erreurs :D

Super, merci les gars :)

Ça montre que ton GCC est mal configuré, comme c'était justement mon cas: Bien configuré, il devrait te donner un warning sur la "déclaration implicite"!

@Médinoc : des petits conseils de config ?
J'ai vu des choses à base de -Wall ou -pedantic-errors.

Je suis débutant en C, j'aime autant que le compileur me garde dans un cadre strict qui m'évite les ambiguités

Pour les principales options, tu trouveras

-Wall active la majorité des erreurs.
-Wextra active la majorité des erreurs non signalées par -Wall
-pedantic (ou -pedantic-errors) signale les écarts à la norme

-Werror ou -Werrors (je ne sais plus la syntaxe) transforme les warning en erreur.
Sachant que les warnings sont la plupart du temps de vraies erreurs.

il peut y avoir un -std=C11 mais je ne sais pas si (comme en C++), c'est la recommandation en C

Tu peux avoir des options d'optimisation (-O2, ou -O3) ou de debug (-g)

Et pour la const-correctness, rajouter -Wwrite-strings à moins que les versions récentes de gcc l'aient ajouté dans l'un des deux autres.