Discussion :

[Guillaume1701]

Bonjour,

Je suis en train d'utiliser la fonction "execvp" mais je me trouve face à un warning qui me donne du fil à retordre.

J'ai le code suivant :

Code :

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

1
2
char *const argv[] = { "/bin/sleep", "20", mullptr};
execv(argv[0], argv);

Si je compile mon code avec std=c++11, je me retrouve avec le warning suivant :

ISO C++ forbids converting a string constant to ‘char*’

Une solution est d'utiliser la fonction "strdup" mais ce n'est pas très "beau". De plus, qui fait un free de la mémoire.

Code :

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

1
2
char *const argv[] = { strdup("/bin/sleep"), strdup("20"), mullptr};
execv(argv[0], argv);

Avez-vous une petite astuce pour éviter ce warning et rendre l'écriture du code plus simple.

Merci d'avance

[foetus]

Et un truc tout bête : c'est ton tableau qui est constant, pas les chaînes de caractères. Et d'ailleurs le compilateur te le dit.

Donc essaye const char* const argv[] = { "/bin/sleep", "20", mullptr};.

[Guillaume1701]

Merci pour la réponse mais cela ne répond pas à mon problème.

Avec le code suivant :

Code :

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

1
2
const char* const argv[] = { "/bin/sleep", "20", nullptr };
execv(argv[0], argv);

J'ai l'erreur suivante à la compilation :

invalid conversion from 'const char* const *' to 'char* const*'

Cela est normal car le prototype de execv

Code :

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

int execv (const char *path, char *const argv[]);

Donc je n'arrive toujours pas à avoir un méthode simple et élégante pour utiliser cette fonction "execv".

[Bousk]

Pourquoi execv devrait-il prendre un tableau de chaînes non constantes en paramètres ?
Soit la fonction est mauvaise et elle prend logiquement un tableau de chaînes constantes, soit la création de argv est incorrecte.

[Guillaume1701]

Pourquoi execv devrait-il prendre un tableau de chaînes non constantes en paramètres ?

C'est une fonction de "unistd.h" (https://linux.die.net/man/3/execv) donc je ne peux pas modifier le prototype.

Soit la fonction est mauvaise et elle prend logiquement un tableau de chaînes constantes, soit la création de argv est incorrecte.

La création du tableau est bien incorrecte pour répondre au prototype de la fonction mais cela est lourd.

[Bousk]

La création du tableau est bien incorrecte pour répondre au prototype de la fonction mais cela est lourd.

Et ? Tu sais quel est le problème mais ne veux pas accepter la solution ? N'es pas capable de voir où est le problème alors que le compilateur te la donne ?
C'est une fonction C, et probablement une vieille, donc d'autant plus souple avec la syntaxe qui est à mon avis daubée. Le 2° paramètre devrait être char const* argv[], mais si on est absolument 500% sûr que c'est un genre d'erreur historique et que les chaînes ne sont réellement pas modifiées, alors c'est un cas typique d'utilisation de const_cast. Et encore mieux je fournirais une fonction qui fait ça en interne.
Sinon, et bien tu crées le paramètre comme attendu, lourd ou pas, t'as pas le choix.

[chrtophe]

Déjà, n'utilises pas argv qui reçoit les arguments de ton programme courant. Créé une variable nommée arguments par exemple.

Ensuite, strdup est une fonction C, y a t'il un équivalent C++ ? je ne sais pas, je ne fais pas de C++.
Si c'est le cas, tu n'as pas à te préoccuper du free, car execv va écraser l'espace mémoire de ton processus par l’exécutable que tu appelles (enfin sauf si il y a une erreur d’exécution). A moins que tu n’exécutes la fonction exec dans un fork.