<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 3, 2021 at 8:48 AM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 9/2/21 2:58 PM, enh wrote:<br>
>     That's why android-ndk-r21d isn't producing this warning. <br>
> <br>
> yeah, sorry: this is for the _host_ toybox (so glibc right now).<br>
<br>
It's not doing it in my host build. Do I have to #define FORTIFY_STUFF somewhere...<br>
<br>
# if __USE_FORTIFY_LEVEL > 0<br>
#  define __wur __attribute_warn_unused_result__<br>
# endif<br>
#else<br>
# define __attribute_warn_unused_result__ /* empty */<br>
#endif<br>
<br>
Apparently so. Sigh.<br></blockquote><div><br></div><div>yeah, all of Android (device and host) builds with _FORTIFY_SOURCE=2.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> adding more of these to bionic has been on the to-do list for years, but turning<br>
> up warnings is always a  hard slog... (see my comment about updating the<br>
> compiler in my previous response!)<br>
<br>
I'm happy for it NOT to produce this warning, I'm just trying to reproduce the<br>
one you wanted to eliminate. :)<br></blockquote><div><br></div><div>iirc the ones that are in bionic aren't tied to __FORTIFY_SOURCE (since that doesn't make much difference for the platform anyway; see above).</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>     Would upgrading to a<br>
>     newer version get it, <br>
> <br>
> not until i add more annotations to bionic, no :-)<br>
>  <br>
>     or is this more aosp-vs-ndk version skew? (And is there an<br>
>     easy way to build external packages with the AOSP prebuilt toolchain?)<br>
> <br>
> if you actually find yourself in a situation where bionic has something that<br>
> isn't in the NDK yet, it's probably easier to update the sysroot (which only<br>
> requires an NDK manifest checkout, not a full AOSP manifest checkout):<br>
> <br>
> <a href="https://android.googlesource.com/platform/ndk/+/master/docs/PlatformApis.md#updating-the-sysroot" rel="noreferrer" target="_blank">https://android.googlesource.com/platform/ndk/+/master/docs/PlatformApis.md#updating-the-sysroot</a><br>
> <<a href="https://android.googlesource.com/platform/ndk/+/master/docs/PlatformApis.md#updating-the-sysroot" rel="noreferrer" target="_blank">https://android.googlesource.com/platform/ndk/+/master/docs/PlatformApis.md#updating-the-sysroot</a>><br>
<br>
Someday I would like to figure out how to build the NDK from source, but I have<br>
zero spare bandwidth at the moment...<br></blockquote><div><br></div><div><a href="https://android.googlesource.com/platform/ndk/+/master/docs/Building.md">https://android.googlesource.com/platform/ndk/+/master/docs/Building.md</a><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Rob<br>
</blockquote></div></div>