tree a44cf68b83b88fba2a2127a936d5c0bdd9f10b45
parent af4de98e69fa27814da1e225565f2a7227c659fe
author Nicolas Lacasse <nlacasse@google.com> 1464983531 -0700
committer Nicolas Lacasse <nlacasse@google.com> 1464990041 +0000

syncbase: Fix memory leak in runRemoteOp.

The runRemoteOp function takes a context and creates a child context.
If the rpc operation is successful, the child context is not cancelled,
and the caller is implicity required to cancel the parent.

In the clock code path, the context passed to runRemoteOp is never
cancelled, and so the child contexts accumulate.

This CL makes the clock create a new context with cancel, and cancels it
in a defer.

Change-Id: I853cb4eb4bd9de35e7be6f15c75cd0940fbfc24c