Skip to content

Codex table cleanup of UB table - #83

Merged
sei-jsible merged 6 commits into
mainfrom
table_cleanup
Jul 21, 2026
Merged

Codex table cleanup of UB table#83
sei-jsible merged 6 commits into
mainfrom
table_cleanup

Conversation

@sei-dsvoboda

Copy link
Copy Markdown
Contributor

One fix: UB 82 lost its text somehow; that is fixed.
Everything else is just formatting (converting HTML to Markdown)

@sei-dsvoboda
sei-dsvoboda requested review from a team and sei-jsible July 9, 2026 21:19
@sei-jsible

Copy link
Copy Markdown
Contributor

This has a merge conflict, but since #75 and #80 also both touch this file, it might be better to hold off on fixing this one until they're merged, so that you don't have to fix it twice.

Comment thread content/4.sei-cert-c-coding-standard/04.back-matter/4.cc-undefined-behavior.md Outdated
Comment on lines -104 to -106
</p></td></tr><tr class="even"><td><div class="content-wrapper"><p>120 <a id="CC.UndefinedBehavior-ub_120"></a></p></div></td><td><p><br />
</p></td><td><p>A pointer returned by the setlocale function is used after a subsequent call to the function, or after the calling thread has exited (7.11.1.1).
</p></td><td><p><a href="/sei-cert-c-coding-standard/rules/environment-env/env34-c">ENV34-C</a>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

UB 120 used to have a link to ENV34-C...

| 117 <a id="CC.UndefinedBehavior-ub_117"></a> | ![](images/icons/emoticons/warning.svg) | The argument to `fesetenv` or `feupdateenv` is neither an object set by a call to `fegetenv` or `feholdexcept` , nor is it an environment macro (7.6.6.3, 7.6.6.4). | |
| 118 <a id="CC.UndefinedBehavior-ub_118"></a> | ![](images/icons/emoticons/warning.svg) | The value of the result of an integer arithmetic or conversion function cannot be represented (7.8.2.1, 7.8.2.2, 7.8.2.3, 7.8.2.4, 7.24.6.1, 7.24.6.2, 7.24.1). | [ERR07-C](/sei-cert-c-coding-standard/recommendations/error-handling-err/err07-c) |
| 119 <a id="CC.UndefinedBehavior-ub_119"></a> | ![](images/icons/emoticons/information.svg) | The program modifies the string pointed to by the value returned by the `setlocale` function (7.11.1.1). | [ENV30-C](/sei-cert-c-coding-standard/rules/environment-env/env30-c) |
| 120 <a id="CC.UndefinedBehavior-ub_120"></a> | | A pointer returned by the setlocale function is used after a subsequent call to the function, or after the calling thread has exited (7.11.1.1). | |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...but now it doesn't.

| 139 <a id="CC.UndefinedBehavior-ub_139"></a> | ![](images/icons/emoticons/warning.svg) | The `va_start` or `va_copy` macro is invoked without a corresponding invocation of the `va_end` macro in the same function, or vice versa (7.16.1, 7.16.1.2, 7.16.1.3, 7.16.1.4). | |
| 140 <a id="CC.UndefinedBehavior-ub_140"></a> | | The `va_arg` macro is invoked when there is no actual next argument, or with a specified type that is not compatible with the promoted type of the actual next argument, with certain exceptions (7.16.1.1). | |
| 141 <a id="CC.UndefinedBehavior-ub_141"></a> | ![](images/icons/emoticons/warning.svg) | The type parameter to the `va_arg` macro is not such that a pointer to an object of that type can be obtained simply by postfixing a `*` (7.16.1.1). | |
| 142 <a id="CC.UndefinedBehavior-ub_142"></a> | | Using a null pointer constant in form of an integer expression as an argument to a . . . function and then interpreting it as a void* or char* (7.16.1.1). | |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the asterisks in void* and char* need escaped so that they show up instead of enabling italics.

@sei-jsible
sei-jsible merged commit 1005f6e into main Jul 21, 2026
@sei-jsible
sei-jsible deleted the table_cleanup branch July 21, 2026 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants