From 7c13d3277ca296ac3fd85bae616085cb8fd8d4e5 Mon Sep 17 00:00:00 2001
From: Nghia Ho <nghiaho12@yahoo.com>
Date: Mon, 6 Jan 2014 20:19:07 +1100
Subject: [PATCH] Fixed a valgrind 'Conditional jump or move depends on
 uninitialised value(s)' on cv::kmeans(...). The original code used
 points(sampleCount, 1, CV_32FC2), which confused generateCentersPP into
 thinking it is a 1 dimensional center, instead of 2. As a result it would set
 only the x variable and leave y unitialised. (cherry picked from commit
 601b7d1dd3a3ec9e8af5df461d43632d98ed3a7a)

---
 samples/cpp/kmeans.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/samples/cpp/kmeans.cpp b/samples/cpp/kmeans.cpp
index 97de6a0a48..b742112059 100644
--- a/samples/cpp/kmeans.cpp
+++ b/samples/cpp/kmeans.cpp
@@ -33,7 +33,7 @@ int main( int /*argc*/, char** /*argv*/ )
     {
         int k, clusterCount = rng.uniform(2, MAX_CLUSTERS+1);
         int i, sampleCount = rng.uniform(1, 1001);
-        Mat points(sampleCount, 1, CV_32FC2), labels;
+        Mat points(sampleCount, 2, CV_32F), labels;
 
         clusterCount = MIN(clusterCount, sampleCount);
         Mat centers(clusterCount, 1, points.type());