<div dir="auto"><div>Sorry and thanks. <br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 27, 2021, 8:35 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The fix looks right, please push it to get the build working again.<br>
<br>
On Fri, Feb 26, 2021 at 11:11 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org" target="_blank" rel="noreferrer">vijay@rtems.org</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> I'm getting build error after this commit.<br>
> ```<br>
> /home/vijay/development/rtems/6/lib/gcc/i386-rtems6/10.2.1/../../../../i386-rtems6/bin/ld:<br>
> ./librtemscpu.a(getgrent.c.57.o): in function `endgrent':<br>
> /home/vijay/rtems-class/rtems/build/i386/pc386/../../../cpukit/libcsupport/src/getgrent.c:109:<br>
> undefined reference to `_Assert_Unused_variable_equals'<br>
> collect2: error: ld returned 1 exit status<br>
> ```<br>
> Looks like assert.h isn't included from getgrent.c . I have attached a<br>
> patch to add the header files and checked that it's building now.<br>
><br>
><br>
> Best regards,<br>
> Vijay<br>
><br>
> On Fri, Feb 26, 2021 at 3:56 PM Joel Sherrill <<a href="mailto:joel@rtems.org" target="_blank" rel="noreferrer">joel@rtems.org</a>> wrote:<br>
> ><br>
> > Module:    rtems<br>
> > Branch:    master<br>
> > Commit:    e5e58da6dfbdfa78ad499c2f52ec53c88538aa1c<br>
> > Changeset: <a href="http://git.rtems.org/rtems/commit/?id=e5e58da6dfbdfa78ad499c2f52ec53c88538aa1c" rel="noreferrer noreferrer" target="_blank">http://git.rtems.org/rtems/commit/?id=e5e58da6dfbdfa78ad499c2f52ec53c88538aa1c</a><br>
> ><br>
> > Author:    Ryan Long <<a href="mailto:ryan.long@oarcorp.com" target="_blank" rel="noreferrer">ryan.long@oarcorp.com</a>><br>
> > Date:      Fri Feb 19 17:30:13 2021 -0500<br>
> ><br>
> > getgrent.c: Fix Unchecked return value error (CID #1459004)<br>
> ><br>
> > CID 1459004: Unchecked return value in endgrent().<br>
> ><br>
> > Closes #4261<br>
> ><br>
> > ---<br>
> ><br>
> >  cpukit/libcsupport/src/getgrent.c | 4 +++-<br>
> >  1 file changed, 3 insertions(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/cpukit/libcsupport/src/getgrent.c b/cpukit/libcsupport/src/getgrent.c<br>
> > index b358cb9..de56430 100644<br>
> > --- a/cpukit/libcsupport/src/getgrent.c<br>
> > +++ b/cpukit/libcsupport/src/getgrent.c<br>
> > @@ -95,6 +95,7 @@ void setgrent(void)<br>
> >  void endgrent(void)<br>
> >  {<br>
> >    grp_context *ctx = grp_get_context();<br>
> > +  int          sc;<br>
> ><br>
> >    if (ctx == NULL)<br>
> >      return;<br>
> > @@ -104,5 +105,6 @@ void endgrent(void)<br>
> >    }<br>
> ><br>
> >    free(ctx);<br>
> > -  pthread_setspecific(grp_key, NULL);<br>
> > +  sc = pthread_setspecific(grp_key, NULL);<br>
> > +  _Assert_Unused_variable_equals(sc, 0);<br>
> >  }<br>
> ><br>
> > _______________________________________________<br>
> > vc mailing list<br>
> > <a href="mailto:vc@rtems.org" target="_blank" rel="noreferrer">vc@rtems.org</a><br>
> > <a href="http://lists.rtems.org/mailman/listinfo/vc" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/vc</a><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>