<div dir="ltr"><div>Hi</div><div><br></div><div>Ryan has wandered into the land of third party code and Coverity issues. It is not a very welcoming land and we need to decide what we want to do as a project. I put one </div><div><br></div><div>In these cases, we have a few patterns to fall back on. There are basically a few choices here:</div><div><br>+ ignore third party code in Coverity. I'm not a fan since we still have code with issues.</div><div><br></div><div>  + Do the minimum which is often just adding (void) in front of the call not checking a return code..Still wrap in #ifdef __rtems__ as below with a more robust solution.  <br></div><div><br>+ Add #ifdef __rtems__ so the original code and rtems changes are side by side. This is more like what we do with libbsd. But I would still tend to use the correct fix and not just slap a (void) in front. If we are going to modify the file, we might as well fix it.<br><br>#ifdef __rtems__<br>   rc = XXX</div><div>   _Assert_Unused_variable_equals'</div><div>#else</div><div>   XXX<br>#endif<br></div><div><br></div><div>Any other options? Opinions. </div><div><br></div><div>--joel</div><div><br></div><div>============================</div> CID 1047324: Unchecked return value in fdt_add_subnode_namelen().<br><br>Closes #4256<br>---<br> cpukit/dtc/libfdt/fdt_rw.c | 2 +-<br> 1 file changed, 1 insertion(+), 1 deletion(-)<br><br>diff --git a/cpukit/dtc/libfdt/fdt_rw.c b/cpukit/dtc/libfdt/fdt_rw.c<br>index 1385425..ceeeb44 100644<br>--- a/cpukit/dtc/libfdt/fdt_rw.c<br>+++ b/cpukit/dtc/libfdt/fdt_rw.c<br>@@ -348,7 +348,7 @@ int fdt_add_subnode_namelen(void *fdt, int parentoffset,<br>             return offset;<br>        /* Try to place the new node after the parent's properties */<br>-    fdt_next_tag(fdt, parentoffset, &nextoffset); /* skip the BEGIN_NODE */<br>+  (void)fdt_next_tag(fdt, parentoffset, &nextoffset); /* skip the BEGIN_NODE */<br>     do {<br>          offset = nextoffset;<br>          tag = fdt_next_tag(fdt, offset, &nextoffset);<br>-- <br>1.8.3.1<br></div>