25

_s functions, such as scanf_s, printf_s seems to be optional standard. MSVC has implemented these functions, but gcc hasn't.

Is there specific reason for not implementing secure functions? Is scanf of glibc secure enough?

10
  • 8
    This may provide some light on the Annex K situation: open-std.org/jtc1/sc22/wg14/www/docs/n1967.htm Jun 6, 2018 at 15:58
  • 10
    Their conclusion: The design of the Bounds checking interfaces, though well-intentioned, suffers from far too many problems to correct. Using the APIs has been seen to lead to worse quality, less secure software than relying on established approaches or modern technologies. More effective and less intrusive approaches have become commonplace and are often preferred by users and security experts alike. Therefore, we propose that Annex K be either removed from the next revision of the C standard, or deprecated and then removed. Jun 6, 2018 at 16:01
  • 1
    Weren't the Annex K functions a Microsoft initiative to begin with? Or at least one that MS was deeply involved with? Jun 6, 2018 at 16:03
  • 1
    @lurker: Where are you using GCC? The problem is actually in the C library. If you're using GCC on Windows and it is set up to know about the functions in the Microsoft C RunTime (CRT), then you'd be able to use them. If you're not on Windows, the functions are not generally available in the local C library. You can find a few add-on implementations, but they're not necessarily complete. See the N1967 paper for more details about what is (un)available. Jun 6, 2018 at 16:15
  • 4
    @JohnBollinger: More or less. The Annex K *_s() functions are closely related to, but critically different from, the *_s() functions that Microsoft added to MSVC. Yes, I believe Microsoft was instrumental in getting the TR 24731-1 report produced, but the result was different in key aspects from what MS had already released in the wilds, and backwards compatibility then became a problem. MS couldn't conform to the standard easily because it would break their backwards compatibility promises. Jun 6, 2018 at 16:24

1 Answer 1

27

The _s functions are optional (Annex K of the C11 standard). They're widely regarded as 'not very beneficial'.

In the answers to my question Do you use the TR-24731 "safe" functions?, you can find information about where there are problems with the standard specification — such as crucial differences between the standard and Microsoft's implementation. TR 24731-1 was a technical report from the C standard committee. The report was incorporated almost verbatim — with an extra, previously omitted, function, memset_s() — in the C11 standard as (optional but 'normative') Annex K. There's also TR 24731-2 for a different set of functions — without the _s suffix. It ran into resistance for a different set of reasons.

Also, there is a proposal before the C Standard Committee that the functions defined in Annex K should be removed from the next revision of the standard:

That paper is a straightforward and compelling read of the reasons why the TR-24731 (*_s()) functions have not been widely implemented.

Key reasons include:

  • The problem is only spotted once, then fixed, and then the *_s() function is unnecessary.
  • This makes it very hard to test the *_s() functions, or the code which uses them.
  • It isn't easy to integrate the new functions into old code (which is where there'd be most benefit).
  • The functions inherently slow down software with extensive but redundant checking.

See the paper for more details. The paper ends with the section:

Suggested Technical Corrigendum

Despite more than a decade since the original proposal and nearly ten years since the ratification of ISO/IEC TR 24731-1:2007, and almost five years since the introduction of the Bounds checking interfaces into the C standard, no viable conforming implementations has emerged. The APIs continue to be controversial and requests for implementation continue to be rejected by implementers.

The design of the Bounds checking interfaces, though well-intentioned, suffers from far too many problems to correct. Using the APIs has been seen to lead to worse quality, less secure software than relying on established approaches or modern technologies. More effective and less intrusive approaches have become commonplace and are often preferred by users and security experts alike.

Therefore, we propose that Annex K be either removed from the next revision of the C standard, or deprecated and then removed.


Annex K was not removed from C17. Two new papers from the standards committee (ISO JTC1/SC22/WG14) discuss Annex K (and are in favour of retaining the functions):


2023-03-05: It looks as though C23 will retain Annex K as an optional part of the standard. However, there is a new function, memset_explicit(), which does the job that memset_s() does — the optimizer is not allowed to skip the code, even if the variable set is not used again.

void *memset_explicit(void *s, int c, size_t n);

Description
2 The memset_explicit function copies the value of c (converted to an unsigned char into each of the first n characters of the object pointed to by s. The purpose of this function is to make sensitive information stored in the object inaccessible380).

380) The intention is that the memory store is always performed (i.e., never elided), regardless of optimizations. This is in contrast to calls to the memset function (7.26.6.1).

10
  • 1
    Honestly the only part of Annex K that should always be implemented is memset_s and only to support security sensitive code. That said it would be nice to have a strtok replacement that's cross platform too.
    – Mgetz
    Jun 27, 2018 at 16:07
  • 1
    Oh I've done that, for strtok it would just be nice to have it be part of the standard rather than a happy happenstance. I do stand by the need for a memset_s that can't be optimized away though.
    – Mgetz
    Jun 27, 2018 at 16:14
  • 3
    @Mgetz: I went and looked at C11 §7.24.6.1 The memset function and §K.3.7.4.1 The memset_s function and the useful difference is indeed the requirement: Unlike memset, any call to the memset_s function shall be evaluated strictly according to the rules of the abstract machine as described in (5.1.2.3). That is, any call to the memset_s function shall assume that the memory indicated by s and n may be accessible in the future and thus must contain the values indicated by c. Jun 27, 2018 at 16:20
  • 3
    That requirement means that calls to memset_s() may not be optimized away, even if the variable being set is not referenced again. That, in turn, allows a programmer to ensure that passwords are nullified when no longer needed, and other similar operations. The extra argument isn't really necessary; the extra semantics related to (non-)optimization most certainly are helpful in a small but significant set of use cases. Jun 27, 2018 at 16:22
  • 2
    @JonathanLeffler: More important than a particular memset_s function, I think, would be a general means of forcing the compiler to synchronize the states of abstract-machine objects and the state of memory seen by the CPU. Using an ordinary memset followed by a force synchronization would be equivalent to memset_s, but security-conscious code needs synchronization for other purposes as well. For example, something like unsigned u = *p; if (u < ARRAY_SIZE) array[u]++; could introduce a security vulnerability if *p gets loaded once for the compare and again for the indexing step.
    – supercat
    Jun 28, 2018 at 19:29

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.