From 37c8a2161e4b349de13642de7f8e33a3e4ec03a3 Mon Sep 17 00:00:00 2001
From: josh <jtpereyda@users.noreply.github.com>
Date: Mon, 24 Aug 2015 12:04:39 -0700
Subject: [PATCH] Fixed bug introduced by refactor. Session.restart_target is
 used (by poll_pedrpc) as if it returns True or False. It actually returned
 None or False, counterintuitively.

---
 sulley/sessions.py | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/sulley/sessions.py b/sulley/sessions.py
index 9ba31c8..ddd8e38 100644
--- a/sulley/sessions.py
+++ b/sulley/sessions.py
@@ -720,6 +720,9 @@ def restart_target(self, target, stop_first=True):
 
         @type  target: session.target
         @param target: Target we are restarting
+
+        @rtype : bool
+        @returns: False if restart failed (such that we know it failed). True otherwise.
         """
 
         # vm restarting is the preferred method so try that first.
@@ -751,6 +754,8 @@ def restart_target(self, target, stop_first=True):
         # pass specified target parameters to the PED-RPC server to re-establish connections.
         target.pedrpc_connect()
 
+        return True
+
     def server_init(self):
         """
         Called by fuzz() on first run (not on recursive re-entry) to initialize variables, web interface, etc...