C++ et exec fonctions

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:

---

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 :

Citation:

---

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:

---

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

Et un truc tout bête :whistle: : 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};.

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

Avec le code suivant :

Code:

---

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

---

J'ai l'erreur suivante à la compilation :

Citation:

---

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

---

Cela est normal car le prototype de execv

Code:

---

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".

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.

Citation:

---

Envoyé par Bousk

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.

Citation:

---

Envoyé par Bousk

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.

Citation:

---

Envoyé par Guillaume1701

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. :weird:

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.