Move serviceinstall to satori

This change is to remove the dependency on a custom version of impacket.
This dependency was necessary because the serviceinstall example inside
impacket did not support custom service/executable names but always
generated a random name for both and a change of the example was very
unlikely to be merged upstream.

Further to that, two minor bugs were fixed
 - satori.smb.SMBClient.create_tunnel() was calling
   satori.tunnel.connect() which has been removed, this has been
   changed to now use the constructor: satori.tunnel.Tunnel()
 - satori.tunnel.Tunnel.serve_forever() was calling self.start
   instead of self.tunnel_thread.start()

Minor style fixes have also been made. Further to that, the
serviceinstall example is not provided under an Apache2 license and
therefore H102 needs to be ignored by tox.

Change-Id: I04f48bf08f9361833cb607faa74ece26a48ea02d