VMCI: Fix "always true condition"

Andy King · gregkh · commit 0e7894dc82bd · 2013-01-17T12:07:39.000-08:00
vmci_send_datagram() returns an int, with negative values indicating failure.
But we store it locally in a u32, which makes comparison of &gt;= 0 useless.
Fixed to use an int.

Reported-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Signed-off-by: Andy King &lt;acking@vmware.com&gt;
Signed-off-by: Dmitry Torokhov &lt;dtor@vmware.com&gt;
Signed-off-by: Greg Kroah-Hartman &lt;gregkh@linuxfoundation.org&gt;
diff --git a/drivers/misc/vmw_vmci/vmci_guest.c b/drivers/misc/vmw_vmci/vmci_guest.c
@@ -78,7 +78,7 @@ bool vmci_guest_code_active(void)
 u32 vmci_get_vm_context_id(void)
 {
 	if (vm_context_id == VMCI_INVALID_ID) {
-		u32 result;
+		int result;
 		struct vmci_datagram get_cid_msg;
 		get_cid_msg.dst =
 		    vmci_make_handle(VMCI_HYPERVISOR_CONTEXT_ID,

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ bool vmci_guest_code_active(void)`
`78`	`78`	`u32 vmci_get_vm_context_id(void)`
`79`	`79`	`{`
`80`	`80`	`if (vm_context_id == VMCI_INVALID_ID) {`
`81`		`- u32 result;`
	`81`	`+ int result;`
`82`	`82`	`struct vmci_datagram get_cid_msg;`
`83`	`83`	`get_cid_msg.dst =`
`84`	`84`	`vmci_make_handle(VMCI_HYPERVISOR_CONTEXT_ID,`