<div dir="ltr">I wish to drop this patch thread and re-submit the patches individually rather than as a group - thanks.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jul 25, 2014 at 5:20 PM, Gary S. Robertson <span dir="ltr"><<a href="mailto:gary.robertson@linaro.org" target="_blank">gary.robertson@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: "Gary S. Robertson" <<a href="mailto:gary.robertson@linaro.org">gary.robertson@linaro.org</a>><br>
<br>
The parse_args() function in realtime test pthread_cond_many contains<br>
a switch statement im which some cases failed to initialize the<br>
function return variable prior to returning... added an init clause<br>
to the definition statement for the return variable.<br>
<br>
Signed-off-by: Gary S. Robertson <<a href="mailto:gary.robertson@linaro.org">gary.robertson@linaro.org</a>><br>
---<br>
...d_many-fix-uninitialized-return-in-option.patch | 31 ++++++++++++++++++++<br>
meta/recipes-extended/ltp/<a href="http://ltp_20140422.bb" target="_blank">ltp_20140422.bb</a> | 1 +<br>
2 files changed, 32 insertions(+)<br>
create mode 100644 meta/recipes-extended/ltp/ltp/0001-pthread_cond_many-fix-uninitialized-return-in-option.patch<br>
<br>
diff --git a/meta/recipes-extended/ltp/ltp/0001-pthread_cond_many-fix-uninitialized-return-in-option.patch b/meta/recipes-extended/ltp/ltp/0001-pthread_cond_many-fix-uninitialized-return-in-option.patch<br>
new file mode 100644<br>
index 0000000..90c9d8e<br>
--- /dev/null<br>
+++ b/meta/recipes-extended/ltp/ltp/0001-pthread_cond_many-fix-uninitialized-return-in-option.patch<br>
@@ -0,0 +1,31 @@<br>
+From 409c00bf4484d40380947fd27329b64ebee29874 Mon Sep 17 00:00:00 2001<br>
+From: "Gary S. Robertson" <<a href="mailto:gary.robertson@linaro.org">gary.robertson@linaro.org</a>><br>
+Date: Fri, 25 Jul 2014 09:41:51 -0500<br>
+Subject: [PATCH] pthread_cond_many: fix uninitialized return in option<br>
+ handling<br>
+<br>
+Some of the cases in the switch statement in parse_args() did not<br>
+initialize the 'handled' variable which is returned by the function.<br>
+Added an initialization to the definition of the variable.<br>
+<br>
+Signed-off-by: Gary S. Robertson <<a href="mailto:gary.robertson@linaro.org">gary.robertson@linaro.org</a>><br>
+---<br>
+ .../realtime/perf/latency/pthread_cond_many.c | 2 +-<br>
+ 1 file changed, 1 insertion(+), 1 deletion(-)<br>
+<br>
+diff --git a/testcases/realtime/perf/latency/pthread_cond_many.c b/testcases/realtime/perf/latency/pthread_cond_many.c<br>
+index f9325b5..1d39100 100644<br>
+--- a/testcases/realtime/perf/latency/pthread_cond_many.c<br>
++++ b/testcases/realtime/perf/latency/pthread_cond_many.c<br>
+@@ -256,7 +256,7 @@ void usage(void)<br>
+<br>
+ int parse_args(int c, char *v)<br>
+ {<br>
+- int handled;<br>
++ int handled = 1;<br>
+ switch (c) {<br>
+ case 'h':<br>
+ usage();<br>
+--<br>
+1.7.9.5<br>
+<br>
diff --git a/meta/recipes-extended/ltp/<a href="http://ltp_20140422.bb" target="_blank">ltp_20140422.bb</a> b/meta/recipes-extended/ltp/<a href="http://ltp_20140422.bb" target="_blank">ltp_20140422.bb</a><br>
index 2ac2187..f86a195 100644<br>
--- a/meta/recipes-extended/ltp/<a href="http://ltp_20140422.bb" target="_blank">ltp_20140422.bb</a><br>
+++ b/meta/recipes-extended/ltp/<a href="http://ltp_20140422.bb" target="_blank">ltp_20140422.bb</a><br>
@@ -27,6 +27,7 @@ SRC_URI = "git://<a href="http://github.com/linux-test-project/ltp.git" target="_blank">github.com/linux-test-project/ltp.git</a> \<br>
file://ltp-Do-not-link-against-libfl.patch \<br>
file://automake-foreign.patch \<br>
file://0001-Allow-mutex-initialization-without-PI-configured.patch \<br>
+ file://0001-pthread_cond_many-fix-uninitialized-return-in-option.patch \<br>
"<br>
<br>
S = "${WORKDIR}/git"<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.9.5<br>
<br>
</font></span></blockquote></div><br></div>