diff options
author | Paul Beesley <paul.beesley@arm.com> | 2019-08-13 11:20:25 +0000 |
---|---|---|
committer | TrustedFirmware Code Review <review@review.trustedfirmware.org> | 2019-08-13 11:20:25 +0000 |
commit | a8ab58e9b291297f3484624a11b24039cb4e1ee8 (patch) | |
tree | b430be4870c855be4c72a65415fcbae560d81d89 /services | |
parent | 3e516bef12fde665bc90682953514c6de3462376 (diff) | |
parent | 9624c0a9e03ea01d3015beafb0a90cd9afd97eb0 (diff) |
Merge changes from topic "jc/coverity-fixes" into integration
* changes:
Fix Coverity #261967, Infinite loop
Fix Coverity #343017, Missing unlock
Fix Coverity #343008, Side affect in assertion
Fix Coverity #342970, Uninitialized scalar variable
Diffstat (limited to 'services')
-rw-r--r-- | services/std_svc/spm/spm_buffers.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/services/std_svc/spm/spm_buffers.c b/services/std_svc/spm/spm_buffers.c index 3e0c9496..79398ba1 100644 --- a/services/std_svc/spm/spm_buffers.c +++ b/services/std_svc/spm/spm_buffers.c @@ -38,6 +38,8 @@ int spm_response_add(uint16_t client_id, uint16_t handle, uint32_t token, struct sprt_response *resp = &(responses[i]); if ((resp->is_valid == 1) && (resp->token == token)) { + spin_unlock(&responses_lock); + return -1; } } |