aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard van der Hoff <richard@matrix.org>2016-09-22 15:09:40 +0100
committerRichard van der Hoff <richard@matrix.org>2016-09-22 15:09:40 +0100
commitcfcee54a817b67fbdb4439e87f6237f8bf3487cc (patch)
treea01247b34267090cccbe0bb0f0d082f1ef57915b
parent047927d822734cffe43adc3f9eae355c12c6db01 (diff)
Handle non-base64 chars in pickle files
-rwxr-xr-xpython/olm/__main__.py44
1 files changed, 18 insertions, 26 deletions
diff --git a/python/olm/__main__.py b/python/olm/__main__.py
index a34d52a..d0fbe17 100755
--- a/python/olm/__main__.py
+++ b/python/olm/__main__.py
@@ -10,6 +10,11 @@ import yaml
from . import *
+def read_base64_file(filename):
+ """Read a base64 file, dropping any CR/LF characters"""
+ with open(filename, "rb") as f:
+ return f.read().translate(None, "\r\n")
+
def build_arg_parser():
parser = argparse.ArgumentParser()
parser.add_argument("--key", help="Account encryption key", default="")
@@ -37,8 +42,7 @@ def build_arg_parser():
def do_keys(args):
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
result = {
"account_keys": account.identity_keys(),
"one_time_keys": account.one_time_keys(),
@@ -55,8 +59,7 @@ def build_arg_parser():
def do_id_key(args):
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
print(account.identity_keys()['curve25519'])
id_key = commands.add_parser("identity_key", help="Get the identity key for an account")
@@ -65,8 +68,7 @@ def build_arg_parser():
def do_one_time_key(args):
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
keys = account.one_time_keys()['curve25519'].values()
key_num = args.key_num
if key_num < 1 or key_num > len(keys):
@@ -93,8 +95,7 @@ def build_arg_parser():
def do_sign(args):
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
with open_in(args.message_file) as f:
message = f.read()
signature = account.sign(message)
@@ -110,8 +111,7 @@ def build_arg_parser():
def do_generate_keys(args):
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
account.generate_one_time_keys(args.count)
with open(args.account_file, "wb") as f:
f.write(account.pickle(args.key))
@@ -132,8 +132,7 @@ def build_arg_parser():
))
sys.exit(1)
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
session = Session()
session.create_outbound(
account, args.identity_key, args.one_time_key
@@ -168,8 +167,7 @@ def build_arg_parser():
))
sys.exit(1)
account = Account()
- with open(args.account_file, "rb") as f:
- account.unpickle(args.key, f.read())
+ account.unpickle(args.key, read_base64_file(args.account_file))
with open_in(args.message_file) as f:
message_type = f.read(8)
message = f.read()
@@ -191,8 +189,7 @@ def build_arg_parser():
def do_session_id(args):
session = Session()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
sys.stdout.write(session.session_id() + "\n")
session_id.set_defaults(func=do_session_id)
@@ -204,8 +201,7 @@ def build_arg_parser():
def do_encrypt(args):
session = Session()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
with open_in(args.plaintext_file) as f:
plaintext = f.read()
message_type, message = session.encrypt(plaintext)
@@ -224,8 +220,7 @@ def build_arg_parser():
def do_decrypt(args):
session = Session()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
with open_in(args.message_file) as f:
message_type = f.read(8)
message = f.read()
@@ -296,8 +291,7 @@ def do_outbound_group(args):
def do_group_encrypt(args):
session = OutboundGroupSession()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
plaintext = args.plaintext_file.read()
message = session.encrypt(plaintext)
with open(args.session_file, "wb") as f:
@@ -306,8 +300,7 @@ def do_group_encrypt(args):
def do_group_credentials(args):
session = OutboundGroupSession()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
result = {
'message_index': session.message_index(),
'session_key': session.session_key(),
@@ -333,8 +326,7 @@ def do_inbound_group(args):
def do_group_decrypt(args):
session = InboundGroupSession()
- with open(args.session_file, "rb") as f:
- session.unpickle(args.key, f.read())
+ session.unpickle(args.key, read_base64_file(args.session_file))
message = args.message_file.read()
plaintext = session.decrypt(message)
with open(args.session_file, "wb") as f: