Re: [PATCH 02/28] cpg test agent: Fix typo in assert

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/06/13 17:02 +0200, Jan Friesse wrote:
Assert should compare rc, instead of setting it.

Signed-off-by: Jan Friesse <jfriesse@xxxxxxxxxx>
---
cts/agents/cpg_test_agent.c |    2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/cts/agents/cpg_test_agent.c b/cts/agents/cpg_test_agent.c
index ed6b766..e8bd914 100644
--- a/cts/agents/cpg_test_agent.c
+++ b/cts/agents/cpg_test_agent.c
@@ -325,7 +325,7 @@ static void read_messages (int sock, char* atmost_str)
		}
	}
	rc = send (sock, big_and_buf, strlen (big_and_buf), 0);
-	assert(rc = strlen (big_and_buf));
+	assert(rc == strlen (big_and_buf));

Oops, I think that was me - blush. Well caught honza/coverity.

}

static qb_loop_timer_handle more_messages_timer_handle;
--
1.7.1

_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss




[Index of Archives]     [Linux Clusters]     [Corosync Project]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Linux Kernel]     [Linux SCSI]     [X.Org]

  Powered by Linux