271
65%
master: 66%

Ran 28 Nov 2016 10:32PM UTC

Jobs 2

Files 91

Run time 69min

Badge

Embed ▾

pending completion

Build # 271

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

13169 of 20230 relevant lines covered (65.1%)

1.4 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	271.1 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)	28 Nov 2016 11:42PM UTC	0	65.08	Travis Job 271.1
2	271.2 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)	28 Nov 2016 10:32PM UTC	0	65.1	Travis Job 271.2

Source Files on build 271

Detailed source file information is not available for this build.

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 271

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 271

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

README BADGES
x