commit bcb45ef5b6c163bd9da948b5d991c76094b495a2
author larsrc <larsrc@google.com> Mon Dec 06 02:52:34 2021 -0800
committer Copybara-Service <copybara-worker@google.com> Mon Dec 06 02:54:12 2021 -0800
tree f6c8a547f6689d5dd4235a38ec698a2101410b2b
parent 402e80ee6ed21311877bd36ca6ee612baf67220b

Avoid NPE in debugPrint() when owner is null

PiperOrigin-RevId: 414390111

src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingContext.java

diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingContext.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingContext.java
index c736a72..02ad695 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingContext.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingContext.java
@@ -276,7 +276,11 @@
     @Override
     public void debugPrint(Printer printer) {
       printer.append("<LinkerInput(owner=");
-      owner.debugPrint(printer);
+      if (owner == null) {
+        printer.append("[null owner, uses old create_linking_context API]");
+      } else {
+        owner.debugPrint(printer);
+      }
       printer.append(", libraries=[");
       for (LibraryToLink libraryToLink : libraries) {
         libraryToLink.debugPrint(printer);