Probleme de passage d'arguments

Version imprimable

16/06/2006, 15h36
izorus

Probleme de passage d'arguments

Salut tout le monde ,

Je me permets de créer ce topic car actuellement je suis en train de coder un petit logiciel de login en mode console permettant de lancer un programme (entre autres) après la vérification du nom d'utilisateur et du password.( il fonctionne sur des sockets principalement)

Il est possible de le lancer avec un maximum de 4 arguments (numéro du port, programme a executer, login et password ) mais lorsque je le lance sans arguments j'ai un problème (de runtime apparemment) et le programme ne démarre pas, bien que je l'ai configuré pour effectuer un traitement par défaut dans ce cas précis.

Je souhaiterais également savoir si il est possible que le programme puisse être lancé par un double clic (et donc avec les paramètres par défaut).

Merci d'avance à ceux qui me répondront !
16/06/2006, 15h39
Swoög

Salut !

Si tu ne nous montre pas ton code, on ne pourra pas te dire ce qui bug...

pour ta seconde question, si ton programme se déroule correctement quand tu le lances sans argument, tu pourras faire des doubles-clics jusqu'à plus soif sans problème ;)

Désolé pour la lisibilité du code et les commentaires, j'ai dû les faire en anglais.
Bien évidemment il n'y a qu'une petite partie du code ,mais le problème devrait venir de là. Re-merci d'avance :)

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
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
 
#include <iostream>
#include <sstream>
#include <winsock2.h>
#include "Data.h"
 
using namespace std;
 
 
int main(int argc,char *argv[])
{
    string portnumber(argv[1]); // Buffer for first argument (port number)
    unsigned short portnum; // port number
    istringstream selectport(portnumber);
    selectport >> portnum; // extract port number from first argument (forgive me, i'm noob ^^)
 
    char *launchprogram=new char[150]; // Buffer for second argument (program to launch)
    launchprogram=argv[2];
 
    data s11,s12; // Objects for save login and password
 
    if (argc == 0 )
    {
             cout << " Default cfg is set. Port number is: 12345 and launched program is: C:\\WINDOWS\\System32\\Cmd.exe " << endl;
             cout << " You can use this syntax: ServLogin.exe portnumber launchedprogram user password" << endl;
             cout << " Example: ServLogin.exe 1337 C:\\WINDOWS\\System32\\Cmd.exe mike greyhat" << endl;
    }
 
    if ((argc != 3) && (argc != 0) && (argc !=5))
    {
             cout <<"\n   -==|   Samael Login Server   |==-\n"<< endl;
             cout << " You can use this syntax: ServLogin.exe portnumber launchedprogram login password" << endl;
             cout << " Or this: ServLogin.exe portnumber launchedprogram" << endl;
             cout << " Example: ServLogin.exe 1337 C:\\WINDOWS\\System32\\Cmd.exe user password" << endl;
             cout << " You can also let the default config" <<endl;
             cout << " Defaut config: "<< endl;
             cout << " Port number is: 12345" << endl;
             cout << " Launched program is: C:\\WINDOWS\\System32\\Cmd.exe"<< endl;
             cout << " Login is: userAlpha" << endl;
             cout << " Password is: passwordAlpha" << endl;
             exit(0);
    }
 
    if (argc == 5)
    {
             s11.set(argv[3],argv[4]); // Set login and password from third and fourth arguments
    }

16/06/2006, 15h55
Swoög

alors :

déjà argc ne sera jamais égale à 0...

puisque argv[0] contient normalement la commande ayant lancer le programme (i.e. le nom de l'exécutable)

en plus, je vois :

Code:

launchprogram=argv[2];

8O !!

ne JAMAIS faire ça pour copier des char* !!

en plus, tu es en C++, utilise des std::string plutôt !

à part ça, je ne peux pas te dire grand chose d'autre...
16/06/2006, 15h57
Matthieu Brucher

Même chose pour la première ligne, tu marques string machin(argv[1]) qui peut être non existant donc valoir n'importe quoi.
T'as passé ton programme au débuggeur au moins ?
16/06/2006, 16h16
izorus

Je n'ai pas l'habitude des string, et donc pas le réflexe de les utiliser il est vrai qu'il faudrait que j'y pense plus souvent!

Et sinon ces lignes de code sont un "premier jet" en quelque sorte je vais de ce pas corriger ces erreurs.
16/06/2006, 16h21
izorus

J'ai un peu honte d'avoir montré un code comme ca, je tacherais de mieux vérifier la "propreté" du code dorénavant avant de poster ^^.
Je ne programme que depuis peu de temps, et ce code a été fait hier soir très tard (ou très tot)
16/06/2006, 16h31
Swoög

Citation:

Envoyé par izorus

J'ai un peu honte d'avoir montré un code comme ca, je tacherais de mieux vérifier la "propreté" du code dorénavant avant de poster ^^.
Je ne programme que depuis peu de temps, et ce code a été fait hier soir très tard (ou très tot)

ça arrive à tout le monde au début, c'est si ça continue après que ça devient grave ;)
19/06/2006, 16h09
izorus

J'ai (enfin) terminé la base de mon serveur, je vous remercie d'ailleurs pour vos conseils, les string m'ont facilité la vie!

Actuellement il me permet de lancer une application lors de mon login ( par exemple activer mon serveur ftp de chez ma copine ^^) mais j'aimerais pouvoir me rediriger vers un shell une fois que je suis loggé(ce qui me permettrait alors de lancer ou de fermer n'importe quelle application)

Existe t'il une facon (assez) simple de le faire ?
Merci d'avance!

ps:Est-ce qu'il existe un outil de test des fuites de mémoires natif windows ?
19/06/2006, 16h35
Médinoc

Test des fuites mémoire natif Windows:
#define _CRTDBG_MAP_ALLOC avant les fichiers d'en-tête
(il semblerait que ça ait aussi un effet sur l'en-tête <new> s'il est inclus après)

et en fin de programme, un appel à _CrtDumpMemoryLeaks().
20/06/2006, 10h32
izorus

Ok je vais aller essayer ca, merci.