On 05/22/2014 03:38 PM, Stephen Rothwell wrote:

Hi Cody,

On Thu, 22 May 2014 15:29:08 -0700 Cody P Schafer [off-list ref] wrote:

quoted

-	*res = be64_to_cpu(result_buffer.result);
+	*res = be64_to_cpu(result_buffer->result);
+	kfree(result_buffer);
+	kfree(request_buffer);
+	return ret;

Why not just fall through here by removing the above 3 lines?

No reason except me not noticing it.

quoted

+
+out_hcall:
+	kfree(result_buffer);
+out_resb:
+	kfree(request_buffer);
+out_reqb:
  	return ret;
  }