FEHLER BEI EINEM ONE TIME PAD

[THE_PYTHON_GUY]

Hallo,
ich habe ein Skript erstellt was erst ein Bild ("test.jpg") verschlüsselt und einen Schlüssel abspeichert (das funktioniert)

Code: Alles auswählen

import os

filename = "test.jpg"

def encrypt(filename):
    to_encrypt = open(filename, "rb").read()
    size = len(to_encrypt)
    key = os.urandom(size)
    with open(filename + ".key", "wb")  as key_out:
        key_out.write(key)
        encrypted = bytes(a ^ b for (a, b) in zip(to_encrypt, key))
        with open(filename, "wb") as encrypted_out:
            encrypted_out.write(encrypted)

encrypt(filename)

das zweite Skript soll das Bild wieder mithilfe des KEYS entschlüsseln, aber es funktioniert nicht.

Code: Alles auswählen

import os

filename = "test.jpg"

def decrypt(filename, key):
    file = open(filename, "rb").read()
    key = open(key, "rb").read()
    decrypted = bytes(a ^ b for (a, b) in zip(file, key))
    with open("d_" + filename, "wb") as decrypted_out:
        decrypted_out.write(decrypted)

decrypt(filename, filename + ".key")

Was ist and dem CODE falsch?

[Dennis89]

Hallo,

was soll denn das '"d_"' in deiner 'with'-Anweisung?

Es wäre auch hilfreich wenn du die vollständige Fehlermeldung postest.

'os' nutzt man übrigens nicht mehr. Pfade erstellt man mit 'pathlib' und puzzelt sie nicht mit '+' zusammen. Wenn du Dateien öffnest, musst du sichergehen dass die wieder geschlossen werden.
Wieso nutzt du nicht überall 'with'? Das sorgt nämlich genau dafür.

Grüße
Dennis

[rogerb]

Was ist and dem CODE falsch?

Also, ich denke es sollte funktionieren.
Wenn ich es ausprobiere wird jedenfalls das Original encrypted und beim decrypten wieder das Original mit dem "d_" erstellt.
Also wenn es bei dir nicht funktioniert, müsstest du mal beschreiben was genau nicht funktioniert.

Was ich aber komisch finde, ist dass, der key die gleiche Byte-Länge wie das Bild haben muss.
Unschön ist dass du unnötigerweise die key-Datei noch geöffnet hast, während auch die Bilddatei geöffnet ist. Außerdem schließt du nicht jedes mal die geöffneten Dateien.
Python vergibt das zwar manchmal, es ist aber nicht sauber. Daher sollte jede Dateioperation in einem with-Kontext laufen um das Schließen zu garantieren.
Ich würde den Code in mehrere Funktionen aufteilen, Lesen und Schreiben von Daten jeweils in getrennten Funktionen und getrennte Funktionen für Encryption und Decryption.

[__deets__]

@rogerb: diese Art der Verschlüsselung mit one-time-Pads sieht genau das vor - der Schlüssel ist zufällig, und mindestens so lange wie der Klartext. Er muss vorher verteilt worden sein. Dann ist das Verfahren mathematisch beweisbar unknackbar. Die Verteilung ist aber natürlich das Problem.

[rogerb]

@__deets__,

guter Hinweis! Dann nehme ich den Kommentar zurück

[Dennis89]

Hm, also wenn ich erst das erste Skript und dann das zweite laufen lasse, dann stört sich Python an dem 'd_'.

Hab ich jetzt die Reihenfolge verwechselt?

Grüße
Dennis

[__blackjack__]

@Dennis89: Was meinst Du mit „es stört sich daran“? Syntaktisch ist es richtig. Und beim schreiben von Dateien sollte man sich den Dateinamen relativ frei aussuchen können. Jedenfalls sollte ein voranstellen von "d_" kein grundsätzliches Problem darstellen.

[Dennis89]

Das Problem bei mir war die Pfadangabe von Windows und das flüchtige Code-überfliegen von mir.

Ich hatte das so stehen:

Code: Alles auswählen

filename = r"C:\Users/Dennis\Pictures\Pfennig.jpg"



def decrypt(filename, key):
    file = open(filename, "rb").read()
    key = open(key, "rb").read()
    decrypted = bytes(a ^ b for (a, b) in zip(file, key))
    with open("d_" + filename, "wb") as decrypted_out:
        decrypted_out.write(decrypted)

decrypt(filename,  filename + ".key")

und habe dann diesen Fehler erhalten:

Code: Alles auswählen

Traceback (most recent call last):
  File "C:\Users\Dennis\AppData\Roaming\JetBrains\PyCharmCE2021.2\scratches\scratch_1.py", line 13, in <module>
    decrypt(filename,  filename + ".key")
  File "C:\Users\Dennis\AppData\Roaming\JetBrains\PyCharmCE2021.2\scratches\scratch_1.py", line 10, in decrypt
    with open("d_" + filename, "wb") as decrypted_out:
OSError: [Errno 22] Invalid argument: 'd_C:\\Users/Dennis\\Pictures\\Pfennig.jpg'

Gelöst habe ich dass dann mit 'pathlib' so:

Code: Alles auswählen

from pathlib import PureWindowsPath

filename = PureWindowsPath("C:/Users/Dennis/Pictures/Pfennig.jpg")


def decrypt(filename, key):
    file = open(filename, "rb").read()
    key = open(key, "rb").read()
    decrypted = bytes(a ^ b for (a, b) in zip(file, key))
    with open(filename.with_stem(f'd_{filename.stem}'), "wb") as decrypted_out:
        decrypted_out.write(decrypted)

decrypt(filename,  filename.parent / (filename.name + ".key"))

Grüße
Dennis

[Sirius3]

PurePath ist falsch, da Du ja IO benutzen willst. Und da Du sogar noch einen PureWindowsPath benutzt hast, ist das Programm nur mit Windows nutzbar.
Du benutzt with_stem aber nicht with_name oder eigentlich with_suffix? Warum?
Path-Objekte mit open zu benutzen ist auch etwas unsinnig.

Code: Alles auswählen

from pathlib import Path

def decrypt(filename):
    data = filename.read_bytes()
    key = filename.with_suffix(filename.suffix + '.key').read_bytes()
    decrypted = bytes(a ^ b for (a, b) in zip(data, key))
    filename.with_name(f"d_{filename.name}").write_bytes(decrypted)

filename = Path("C:/Users/Dennis/Pictures/Pfennig.jpg")
decrypt(filename)

[Dennis89]

PurePath ist falsch, da Du ja IO benutzen willst. Und da Du sogar noch einen PureWindowsPath benutzt hast, ist das Programm nur mit Windows nutzbar.

Das Programm, das ich gepostet habe sollte keines Falls eine Lösung oder Verbesserung sein. Ich habe nur eine Möglichkeit gesucht, den Code des TE bei mir ohne Fehler auszuführen.

Du benutzt with_stem aber nicht with_name oder eigentlich with_suffix? Warum?

'with_suffix' hatte ich drin, aber das hat mir 'jpg' durch 'key' ersetzt und nicht 'key' hinzugefügt. Dann bin ich gleich davon abgewichen. Mittlerweile weis ich, das ich mich bei dem Versuch verstrickt hatte.

Path-Objekte mit open zu benutzen ist auch etwas unsinnig.

Hier habe ich jetzt bewusst das 'open' drin gelassen, weil ich den ursprünglichen Code nicht ändern wollte, sondern nur bei mir zu Testzwecken ausführen und einen eventuellen Fehler zu entdecken.

Zum Schluss, das hätte eigentlich an den Anfang gehört: Danke für deine Verbesserungsvorschläge.

Grüße
Dennis