Re: [Xen-devel] [PATCH v2] libxc: fix leak in xc_offline_page error path

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Harmandeep Kaur writes ("[PATCH v2] libxc: fix leak in xc_offline_page error 
path"):
> Avoid leaking the mapping of the m2p in one of the possible failure cases.
> 
> Coverity CID 1351225
> 
> Signed-off-by: Harmandeep Kaur <write.harmandeep@xxxxxxxxx>

Thanks.

Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

It is conventional, when you post a v2, to include the
still-applicable formal review tags.

Sometimes it is a matter of judgement, to decide whether an Acked-by
or Reviewed-by, still stands.  But in this case Dario said:

  The code looks ok to me, so, with or without the subject/changelog
  improvements:
  
  Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>

So, he has clearly indicated that your revised version should be
tagged:

  Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>

You would put that right after the Reviewed-by.


However, there is no need to resend your patch for this.  It is normal
for a committer to search the thread for acks to the patch which
arrive after it was posted, and fold them in.

In this case since the Subject has changed, someone needs to make sure
that that search turns up the review, but this message of mine will
serve that purpose.


So, having said all that, thanks for your contribution :-).

Regards,
Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel