This patch does the following:
1. Fixing instances of finished() being called after error() in smb slave.
2. Changing behaviour of the sftp/smb slave to conform to what is stated in D23194. In particular, the sftp slave should not call finished() in the open()/read()/write()/seek() methods.