On Mon, Dec 11, 2017 at 10:27:58AM +0000, Frediano Ziglio wrote: > Coverity complaint that this field should be protected by > a mutex as other accesses are with the mutex locked. > Use atomic operation. Not in an hot path in any case. > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > --- > server/stat-file.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/server/stat-file.c b/server/stat-file.c > index 2797fd73..45dece65 100644 > --- a/server/stat-file.c > +++ b/server/stat-file.c > @@ -190,7 +190,7 @@ stat_file_add_counter(RedStatFile *stat_file, StatNodeRef parent, const char *na > return NULL; > } > node = &stat_file->stat->nodes[ref]; > - node->flags |= SPICE_STAT_NODE_FLAG_VALUE; > + __sync_or_and_fetch(&node->flags, SPICE_STAT_NODE_FLAG_VALUE); or g_atomic_int_or ? Christophe
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel