Bogus assert

implementation can indeed be empty when cppu::writeSharedLibComponentInfo calls
getEnvironmentFromModule(mod, curEnv, "", ""), which in the final if-branch
calls getEnvironment with implementation = "".  Happens when adding desktop/qa's
test-active.oxt.  Regression introduced with
ae3a0c8da50b36db395984637f5ad74d3b4887bc "Add .component <implementation
constructor='...' feature."

Change-Id: Ia70958e4aa18d378ff35063bdf6187f6e073b765

1 files changed, 12 insertions, 10 deletions

diff --git a/cppuhelper/source/shlib.cxx b/cppuhelper/source/shlib.cxx
index e4f5c05efb6c..7e334e7627ed 100644
--- a/cppuhelper/source/shlib.cxx
+++ b/cppuhelper/source/shlib.cxx
@@ -40,17 +40,19 @@
 css::uno::Environment cppuhelper::detail::getEnvironment(
     rtl::OUString const & name, rtl::OUString const & implementation)
 {
-    assert(!implementation.isEmpty());
     rtl::OUString n(name);
-    static char const * log = std::getenv("UNO_ENV_LOG");
-    if (log != nullptr && *log != 0) {
-        rtl::OString imps(log);
-        for (sal_Int32 i = 0; i != -1;) {
-            rtl::OString imp(imps.getToken(0, ';', i));
-            //TODO: this assumes UNO_ENV_LOG only contains ASCII characters:
-            if (implementation.equalsAsciiL(imp.getStr(), imp.getLength())) {
-                n += ":log";
-                break;
+    if (!implementation.isEmpty()) {
+        static char const * log = std::getenv("UNO_ENV_LOG");
+        if (log != nullptr && *log != 0) {
+            rtl::OString imps(log);
+            for (sal_Int32 i = 0; i != -1;) {
+                rtl::OString imp(imps.getToken(0, ';', i));
+                //TODO: this assumes UNO_ENV_LOG only contains ASCII characters:
+                if (implementation.equalsAsciiL(imp.getStr(), imp.getLength()))
+                {
+                    n += ":log";
+                    break;
+                }
             }
         }
     }