diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2020-12-08 16:38:44 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2020-12-10 16:27:35 +0100 |
| commit | 4f0c70fb5554325e0cc2129741175bf07de22029 (patch) | |
| tree | f6fc98fa67183594f1173c089979cc5f2ac8ff4a /idl | |
| parent | 65b1152f5cc67067b66945d8bb2db009a3cb554e (diff) | |
Avoid calling OString ctor with null pointer
...in preparation of potential future changes from using OString to using
std::string_view, where OString has an undocumented feature of allowing
construction from a null pointer.
This is mostly the result of a manual audit of potentially problematic getenv
calls across the code base. But there can be other problematic places too, like
the xmlGetProp call in tools/source/xml/XmlWalker.cxx. To identify those,
rtl_{string,uString}_newFromStr aborts now in non-production debug builds when a
null pointer is passed(and all places that hit with a full `make check
screenshot` have been addressed here). Once we are confident that all
problematic places have been identified, we should drop support for the
undocumented feature (see the TODO in sal/rtl/strtmpl.cxx).
Change-Id: I595cc6d4f1cda74add2a3db171323f817d362b08
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107430
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'idl')
| -rw-r--r-- | idl/source/prj/command.cxx | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/idl/source/prj/command.cxx b/idl/source/prj/command.cxx index 35f6e6c28e87..29b01d11219a 100644 --- a/idl/source/prj/command.cxx +++ b/idl/source/prj/command.cxx @@ -277,7 +277,8 @@ SvCommand::SvCommand( int argc, char ** argv ) aList.clear(); - OString aInc(getenv("INCLUDE")); + auto const env = getenv("INCLUDE"); + OString aInc(env == nullptr ? "" : env); // append include environment variable if( aInc.getLength() ) { |