diff options
author | Ingela Anderton Andin <[email protected]> | 2012-08-29 11:50:39 +0200 |
---|---|---|
committer | Ingela Anderton Andin <[email protected]> | 2012-08-29 17:49:17 +0200 |
commit | 1429ac4b0a76c73224382a32ecd28ee89f5327ce (patch) | |
tree | 073ff21d28fdfe0fa3d774d8707f7de8f8f18bbb /lib/ssh/src/ssh_file.erl | |
parent | 0080be4a20c094ef45dde842ace64f16d5dc22b2 (diff) | |
download | otp-1429ac4b0a76c73224382a32ecd28ee89f5327ce.tar.gz otp-1429ac4b0a76c73224382a32ecd28ee89f5327ce.tar.bz2 otp-1429ac4b0a76c73224382a32ecd28ee89f5327ce.zip |
ssh: Increase robustness
If a line of single line of the authorized_key file och known_hosts file
is incorrectly formated ignore it.
Diffstat (limited to 'lib/ssh/src/ssh_file.erl')
-rw-r--r-- | lib/ssh/src/ssh_file.erl | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/lib/ssh/src/ssh_file.erl b/lib/ssh/src/ssh_file.erl index d05fa8e09a..a6b82a7a13 100644 --- a/lib/ssh/src/ssh_file.erl +++ b/lib/ssh/src/ssh_file.erl @@ -232,7 +232,7 @@ lookup_host_key_fd(Fd, Host, KeyType) -> eof -> {error, not_found}; Line -> - case public_key:ssh_decode(Line, known_hosts) of + case ssh_decode_line(Line, known_hosts) of [{Key, Attributes}] -> handle_host(Fd, Host, proplists:get_value(hostnames, Attributes), Key, KeyType); [] -> @@ -240,6 +240,13 @@ lookup_host_key_fd(Fd, Host, KeyType) -> end end. +ssh_decode_line(Line, Type) -> + try + public_key:ssh_decode(Line, Type) + catch _:_ -> + [] + end. + handle_host(Fd, Host, HostList, Key, KeyType) -> Host1 = host_name(Host), case lists:member(Host1, HostList) and key_match(Key, KeyType) of @@ -285,7 +292,7 @@ lookup_user_key_fd(Fd, Key) -> eof -> {error, not_found}; Line -> - case public_key:ssh_decode(Line, auth_keys) of + case ssh_decode_line(Line, auth_keys) of [{AuthKey, _}] -> case is_auth_key(Key, AuthKey) of true -> |