266
65%
master: 66%

Ran 28 Nov 2016 07:29PM UTC

Jobs 2

Files 91

Run time 2min

Badge

Embed ▾

pending completion

Build # 266

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

13168 of 20184 relevant lines covered (65.24%)

1.4 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	266.1 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)	28 Nov 2016 07:29PM UTC	0	65.24	Travis Job 266.1
2	266.2 (COVERALLS_TOKEN=mwTn1pOFqEOUT13vylZNHq53NanoMznO7)	28 Nov 2016 07:31PM UTC	0	65.24	Travis Job 266.2

Source Files on build 266

Detailed source file information is not available for this build.

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 266

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 266

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

README BADGES
x