Bien le bonjour!

Je suis entrain en ce moment de créer une simulation de programme de crack de mot de passe par brute force avec dictionnaire.

Pas de panique, je n'ai aucune mauvaise intention, tout cela est un exercice purement théorique, et je n'ai de toute façons pas du tout le niveau pour créer un tel programme fonctionnel. (c'est l'exercice du pendu qui m'a donné cette idée)

seulement j'ai un problème avec une boucle, voici la fonction:

void BruteForce(char motDePasse)
{
char chaineNombres[3];
char Mot[100] = {0};
char Stockage[100];
FILE *fichierTest = NULL;
fichierTest = fopen("test.txt", "r");
    if (fichierTest != NULL)
    {

    rewind(fichierTest);
        do
        {
            fgets(Mot, 100, fichierTest);
            Mot[strlen(Mot) - 1] = '\0';
            int i = 0;
            do
            {
                //on crée stockage, copie de Mot, que l'on utilisera pour la comparaison
                //Cela évite de devoir rechercher le mot a chaque réinitialisation
                strcpy(Stockage, Mot);

                //on récupere notre chaine de nombres

                ChaineNombres(chaineNombres , i);
                i++;

                //on concatène les deux chaines

                strcat(Stockage, chaineNombres);
                printf("%s\n", Stockage);
            } while (i < 100);
        } while (strcmp(Stockage, motDePasse) != 0 && fichierTest != NULL);
    }
    else
        {
        printf ("Probleme d'ouverture du dictionnaire");
        }

}

La fonction prend comme paramètre le mot de passe à trouver (entré dans une chaîne par le "joueur", plus haut dans le programme)
Ensuite, la fonction pioche le premier mot de la liste du document "test.txt" et l'intègre dans la chaine "Mot"

Puis la fonction ChaineNombres reçois un nombre entre 1 et 99  et renvois une chaine contenant ce nombre reçu avec un \0 a la fin

ex: pour une entrée de 54, on obtient la chaine suivante:

     chaineNombres[0] = '5'

     chaineNombres[1] = '4'

     chaineNombres[2] = '\0'

Puis le programme concatène Mot (devenu StockageMot ici) et ChaineNombre pour obtenir pour chaque mot de la liste 99 variantes avec des nombres a la fin de 0 a 99

La fonction continue tant que le MotDePasse n'a pas été trouvé, et tant qu'il reste des mots dans le dico.

Mon problème est le suivant: La boucle va prendre le premier mot et lui associer tous les nombres possibles comme prévu, mais ne va pas passer au mot suivant après que ma boucle Do While se termine...

J'espère que mon problème est assez clair! 

Merci et bonne journée!

-
Edité par AlexandrePlouvier2 23 juin 2020 à 15:23:06

Bonjour !

AlexandrePlouvier2 a écrit:

La fonction prend comme paramètre le mot de passe à trouver (entré dans une chaîne par le "joueur", plus haut dans le programme)

Du coup il y a déjà une erreur dans le paramètre : tu as mis un caractère tout court. Il faudrait plutôt :

void BruteForce(char motDePasse[])

Après, il y a peut-être d'autres erreurs.

- Ligne 15 : tu ajoutes le caractère nul en fin de chaîne. Tu es sûr qu'il n'y est pas déjà ? D'autant que tu utilises pour ça la fonction 'strlen', qui se base justement sur celui-ci pour calculer la longueur.

- Ligne 21 : inutile. La variable 'Mot' contient le mot, et ensuite tu lui ajoutes les nombres entre 0 et 99 grâce à la ligne 30, mais 'Mot' n'est jamais perdu. Pour faire cette opération, tu pouvais aussi utiliser la fonction 'sprintf' :

    sprintf(Stockage, "%s%02d", Mot, i);

Ça remplace l'enchaînement de 'strcpy', 'ChanîneNombre' et 'strcat'.

Mais comme les boucles sont écrites, tu testes si le mot de passe est égal au mot sans les nombres (une fois corrigée l'erreur du paramètre). Elles sont donc mal écrites. Une méthode possible serait de procéder ainsi :

mdp_trouvé = Faux
Faire :
{   Lire Mot dans le fichier
    Pour i = 0 à 99 :
        Construire Stockage := Mot + 'i'
        Si Stockage = Mot_de_Passe alors :
            mdp_trouve = Vrai
            break
    Fin Pour
    // Ici, soit on est sorti du "Pour" parce qu'on a trouvé le mot de passe,
    // soit on est sorti parce qu'on a parcouru tous les nombres sans rien trouver
} Tant Que Non mdp_trouvé
// Ici, soit on est sorti du "Faire Tant Que" parce qu'on a trouvé le mot de passe,
// soit on est sorti parce qu'on a parcouru tout le fichier sans rien trouver
Si mdp_trouvé alors :
    (conclure...)

-
Edité par robun 23 juin 2020 à 16:02:17

Ouaou bah c'était bien ça le problème merci beaucoup! Apparemment c'était la seule erreur!

Je fais beaucoup d'erreurs d'inattentions dans ce genre et j'ai du mal a les trouver dans mon programme à la relecture..

C'est super d'avoir la possibilité de faire relire par des gens plus compétents, aussi je vous remercie de prendre de votre temps pour lire et répondre!

Bien cordialement!