[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: New santizer warning in KF 5.98 headers
From:       Michael Reeves <reeves.87 () gmail ! com>
Date:       2023-01-10 22:45:26
Message-ID: CANp6eRkDwEJZuXb6kM8aLj2sOn-JTkXAcURXzhB-uhVeOhvTHQ () mail ! gmail ! com
[Download RAW message or body]

Thanks. I would say your right there this would definitely have caught
someone's attention if didn't work in practice with what kde needs.
Santizers are by design quite pedantic as serves there purpose well.

On Tue, Jan 10, 2023 at 5:34 PM Albert Astals Cid <aacid@kde.org> wrote:

> El dimarts, 10 de gener de 2023, a les 22:49:43 (CET), Michael Reeves va
> escriure:
> > /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64: runtime error:
> > load of value 4294967295, which is not a valid value for type
> > 'Qt::ConnectionType'
> >
> > SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior
> > /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64 in
> >
> > The issue stems for assigning an int to a enum which is internally
> > considered unsigned and possibly smaller than the four byte int. If this
> is
> > doing what we expect than I need a way to shut off the warning.
>
> That code has been there since May last year, so not exactly "new".
>
> Given it doesn't seem to be crashing it would seem it's one of those
> "undefined
> but it works in all the compilers we care about".
>
> Of course patches to make the sanitizer are really welcome :)
>
> Cheers,
>   Albert
>
>
>
>

[Attachment #3 (text/html)]

<div dir="ltr">Thanks. I would say your right there this would definitely have caught \
someone&#39;s attention if didn&#39;t work in practice with what kde needs. Santizers \
are by design quite pedantic as serves there purpose well.<br></div><br><div \
class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 10, 2023 at 5:34 PM \
Albert Astals Cid &lt;<a href="mailto:aacid@kde.org">aacid@kde.org</a>&gt; \
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">El dimarts, 10 de \
gener de 2023, a les 22:49:43 (CET), Michael Reeves va <br> escriure:<br>
&gt; /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64: runtime error:<br>
&gt; load of value 4294967295, which is not a valid value for type<br>
&gt; &#39;Qt::ConnectionType&#39;<br>
&gt; <br>
&gt; SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior<br>
&gt; /usr/include/KF5/KConfigWidgets/kstandardaction.h:261:64 in<br>
&gt; <br>
&gt; The issue stems for assigning an int to a enum which is internally<br>
&gt; considered unsigned and possibly smaller than the four byte int. If this is<br>
&gt; doing what we expect than I need a way to shut off the warning.<br>
<br>
That code has been there since May last year, so not exactly &quot;new&quot;.<br>
<br>
Given it doesn&#39;t seem to be crashing it would seem it&#39;s one of those \
&quot;undefined <br> but it works in all the compilers we care about&quot;.<br>
<br>
Of course patches to make the sanitizer are really welcome :)<br>
<br>
Cheers,<br>
   Albert<br>
<br>
<br>
<br>
</blockquote></div>



[prev in list] [next in list] [prev in thread] [next in thread]