[PATCH] fix cid 1350009

Michael Adam obnox at samba.org
Wed Feb 10 10:03:51 UTC 2016


On 2016-02-10 at 09:46 +0200, Uri Simchoni wrote:
> It's better, but having carefully looked at it now, I think you also want to
> set speed to 0 on all failure paths, or report failure to caller and let it
> intelligently deal with it.

Well, I also thought about it.
Currently the pattern is: only set speed on success.
Otherwise leave speed untouched. In the current caller
this means that the default is used.

Of course we could propagate the error, but it seemed
the most simple this way.

What specific problem do you see with the current way?

Thanks - Michael

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20160210/55de06f4/signature.sig>


More information about the samba-technical mailing list