Skip to content
This repository was archived by the owner on Jun 20, 2024. It is now read-only.

Commit faf3ed9

Browse files
Merge pull request #70 from SethMichaelLarson/empty-password
Raise more descriptive error with zero-length password.
2 parents 570cbea + 5e1e8f4 commit faf3ed9

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

docs/source/changelog.rst

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,8 @@ Change log
33

44
master
55

6+
* IGuest.create_session() now raises a more descriptive error if
7+
not able to connect with a zero-length password. (@SethMichaelLarson PR #70)
68
* Add sys.executable-derived paths in list to check for vboxapi (@SethMichaelLarson PR #69)
79
* Fix IGuestProcess.execute() on Python 3.x (@SethMichaelLarson PR #58)
810
* Fix errors to not output on Windows platforms. (@SethMichaelLarson PR #57)

virtualbox/library_ext/guest.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@ def create_session(self, user, password, domain='', session_name='pyvbox',
2121
break
2222
time.sleep(0.1)
2323
else:
24+
if len(password) == 0:
25+
raise SystemError('GuestSession failed to start. Could be because '
26+
'of using an empty password.')
2427
raise SystemError("GuestSession failed to start")
2528
if timeout_ms != 0:
2629
# There is probably a better way to to this?

0 commit comments

Comments
 (0)