Nun wollte ich 2 Funktionen schreiben die mir die Verschlüsselung in String formatieren:

void encrypt(std::string &text, std::string password)
{
    char* data = const_cast<char*> (text.c_str());
    char* pw   = const_cast<char*> (password.c_str());

    aes AES;
    AES.encrypt(&data, text.length(), pw);
    text = data;

    delete [] data;
    delete [] pw;
}

void decrypt(std::string& text, std::string password)
{
    char* data = const_cast<char*> (text.c_str());
    char* pw   = const_cast<char*> (password.c_str());

    aes AES;
    AES.decrypt(&data, text.length(), pw);
    text = data;

    delete [] data;
    delete [] pw;
}

Das klappt alles ohne Fehler aber wen ich mehr als 30 Zeichen verschlüsseln will bricht er ab und schreibt nur die 30 Zeichen(Und ein wenig Datenmüll -.-)

Liegt dass an const_cast<char*> ?

Bist du des Wahnsinns? Überleg mal, was du da tust:

1. const_cast heißt soviel wie: "Da ist ein Fehler in dieser fremden Bibliothek die ich benutzen möchte, aber ich weiß es besser und das ist ganz sicher richtig so."
Bist du dir wirklich sicher, dass du es besser als die Standardbibliothek weißt?
2. &data. Die Adresse eines Pointers. Sicher, dass du das möchtest?
3. Das delete ist da so etwas von falsch, da fällt mir noch nicht einmal ein lässiger Spruch zu ein. Du gibst da Teile des Strings frei!

Ich halte es für ein kleines Wunder, dass da überhaupt auch nur irgendetwas funktioniert und das nicht immer abstürzt.

Liegt dass an const_cast<char*> ?

Auch. Aber es liegt vor allem daran, dass du nicht verstehst, was du da tust. Das sind ganz grobe Fehler. Bevor du fremde Bibliotheken nutzt, musst du mindestens die Grundlagen verstehen. Besonders wenn du nur reines C++ kannst und nun eine C-Schnittstelle benutzen musst.

Probier mal:

void decrypt(const std::string& text, const std::string& password)
{
    char* data = new char[text.length() + 1];
    char* pw = new char[password.length() + 1]; // Zu faul um nachzugucken ob das verändert wird.
    std::strcpy(data, text.c_str());
    std::strcpy(pw, password.c_str());

    aes AES;
    AES.decrypt(&data, text.length(), pw);

    delete[] data;
    delete[] pw;
}

Trotzdem würde ich dir raten den ganzen Kram erst mal ordentlich umzuschreiben.

Bist du dir wirklich sicher, dass du es besser als die Standardbibliothek weißt?
2. &data. Die Adresse eines Pointers. Sicher, dass du das möchtest?

Wenn du dir den Code anguckst, ist das tatsächlich korrekt. Auch wenn es schon beim Lesen weh tut. Es wird noch schlimmer wenn du den AES Code siehst.

SeppJ schrieb:

Bist du dir wirklich sicher, dass du es besser als die Standardbibliothek weißt?
2. &data. Die Adresse eines Pointers. Sicher, dass du das möchtest?

Wenn du dir den Code anguckst, ist das tatsächlich korrekt. Auch wenn es schon beim Lesen weh tut. Es wird noch schlimmer wenn du den AES Code siehst.

Oh, ich seh's, jetzt wo ich mir den Code von Swordfish angucke. Aber das liegt an Swordfishs Code. Der ist ja grauenhaft! Ich kann schon beim groben drüberschauen ein paar dicke Fehler sehen (und damit meine ich nicht den schlechten Stil, sondern echte Fehler).

@Threadersteller: Benutz doch lieber etwas etabliertes wie Crypto++, anstatt Lösungen aus zufälligen Forenbeiträgen von irgendwelchen Leuten aus dem Internet.