At Mon, 11 Aug 2014 18:11:44 +0800, Ruoyu wrote: > > Nothing wrong with fixing it, alhough I don't know exactly > what will be happened if not patching it. > > Signed-off-by: Ruoyu <lian...@ucweb.com> > --- > sheep/vdi.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/sheep/vdi.c b/sheep/vdi.c > index 05cae7b..77dc253 100644 > --- a/sheep/vdi.c > +++ b/sheep/vdi.c > @@ -372,11 +372,12 @@ static bool add_new_participant(struct vdi_state_entry > *entry, > return true; > } > > - idx = entry->nr_participants++; > + idx = entry->nr_participants; > memcpy(&entry->participants[idx], owner, sizeof(*owner)); > entry->participants_state[idx] = > is_modified(entry) ? > SHARED_LOCK_STATE_INVALIDATED : SHARED_LOCK_STATE_SHARED; > + entry->nr_participants++;
I think this patch doesn't change the function. Does this fixes a bug? Thanks, Hitoshi -- sheepdog mailing list sheepdog@lists.wpkg.org http://lists.wpkg.org/mailman/listinfo/sheepdog