apt/apt-pkg/contrib/configuration.cc, branch 2.9.0

Modernize standard library includes

2024-02-20T12:49:04Z

This was automated with sed and git-clang-format, and then I had to fix up the top of policy.cc by hand as git-clang-format accidentally indented it by two spaces.

Prevent infinite loop in `ReadConfigFile`

2023-12-29T06:40:40Z

Break the loop on failure. Without this, the function goes into an infinite loop if `FName` is a directory.

Address statements of public domain

2023-03-06T09:57:55Z

Spelling fixes

2021-11-27T10:22:38Z

configuration: Add missing #include

2021-02-24T09:47:51Z

As user "DaOfficialRolex" on GitHub pointed out: This is needed to allow for APT on iOS to compile correctly. If not included the two following errors happen while compiling APT. ~/apt/apt-pkg/contrib/configuration.cc:900:44: error: constexpr variable cannot have non-literal type 'const std::array' constexpr std::array magicComments { "clear"_sv, "include"_sv, "x-apt-configure-index"_sv }; ^ ~/apt/apt-pkg/contrib/configuration.cc:900:44: error: implicit instantiation of undefined template 'std::__1::array' /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/__tuple:219:64: note: template is declared here template struct _LIBCPP_TEMPLATE_VIS array; ^

Avoid overstepping bounds in config file parsing

2021-02-03T16:43:13Z

Our configuration files are not security relevant, but having a parser which avoids crashing on them even if they are seriously messed up is not a bad idea anyway. It is also a good opportunity to brush up the code a bit avoiding a few small string copies with our string_view.

Retire and deprecate _strtabexpand

2021-02-03T16:36:45Z

If the Configuration code calling this was any indication, it is hard to use – and even that monster still caused heap-buffer-overflow errors, so instead of trying to fix it, lets just use methods which are far easier to use. The question why this is done at all remains, but is left for another day as an exercise for the reader.

Fail ConfigDir reading if directory listing failed

2021-02-03T16:36:45Z

We were printing an error and hence have non-zero exit code either way, but API wise it makes sense to have this properly reported back to the caller to propagate it down the chain e.g. while parsing #include stanzas.

Fix some style warnings from cppcheck

2019-11-26T11:36:46Z

Unused variable, std::algorithms instead of raw for-loops. There should be no observeable difference in behaviour. Reported-By: cppcheck Gbp-Dch: Ignore

configuration: Compare size first during lookup

2018-12-26T20:24:20Z