1
65%
master: 66%

Ran 28 Nov 2016 11:42PM UTC

Files 91

Run time 5s

Badge

Embed ▾

Committed 28 Nov 2016 10:24PM UTC coverage: 65.082% (-0.2%) from 65.239%

Job # COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7

Build Type

push

travis-ci

Committed by

kaccardi

Commit Message

ciao-controller: datastore: do not use defer to unlock mutex in AddPool()

In the error path for AddPool(), we need to delete the added pool
if there was a failure. This function does it's own locking of the
pools mutex, so in order to ensure that we don't deadlock, we need
to first unlock the mutex prior to calling. Because of that, we cannot
use a defer on the Unlock() call. Remove the defer statement, and
add the explicit Unlock() call to all the error paths in the AddPool()
function.

Signed-off-by: Kristen Carlson Accardi <kristen@linux.intel.com>

Run Details

13166 of 20230 relevant lines covered (65.08%)

0.7 hits per line

01org / ciao / 271 / 1
65%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 271.1 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)

01org / ciao / 271 / 1 65% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 271.1 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)

01org / ciao / 271 / 1
65%
master: 66%

README BADGES
x