Add RLC-only SideEffectFree stubs and tests

checker/src/main/java/org/checkerframework/checker/rlccalledmethods/RLCCalledMethodsChecker.java

@@ -19,7 +19,7 @@
  * CalledMethodsChecker} used as a subchecker in the ResourceLeakChecker, and never independently.
  * Runs the MustCallChecker as a subchecker in order to share the CFG.
  */
-@StubFiles("IOUtils.astub")
+@StubFiles({"IOUtils.astub", "log4j.astub"})
 public class RLCCalledMethodsChecker extends CalledMethodsChecker {
 
   /** Creates a RLCCalledMethodsChecker. */

checker/src/main/java/org/checkerframework/checker/rlccalledmethods/jdk.astub

@@ -0,0 +1,19 @@
+package java.lang;
+
+import org.checkerframework.checker.lock.qual.GuardSatisfied;
+import org.checkerframework.dataflow.qual.SideEffectFree;
+
+public interface AutoCloseable {
+  @SideEffectFree
+  void close(@GuardSatisfied AutoCloseable this) throws Exception;
+}
+
+package java.io;
+
+import org.checkerframework.checker.lock.qual.GuardSatisfied;
+import org.checkerframework.dataflow.qual.SideEffectFree;
+
+public interface Closeable extends AutoCloseable {
+  @SideEffectFree
+  public void close(@GuardSatisfied Closeable this) throws IOException;
+}

checker/src/main/java/org/checkerframework/checker/rlccalledmethods/log4j.astub

@@ -0,0 +1,212 @@
+import org.checkerframework.dataflow.qual.SideEffectFree;
+
+// Log4j 1.x API stubs (package org.apache.log4j).
+package org.apache.log4j;
+
+class Category {
+  @SideEffectFree public void debug(Object message);
+  @SideEffectFree public void debug(Object message, Throwable t);
+  @SideEffectFree public void info(Object message);
+  @SideEffectFree public void info(Object message, Throwable t);
+  @SideEffectFree public void warn(Object message);
+  @SideEffectFree public void warn(Object message, Throwable t);
+  @SideEffectFree public void error(Object message);
+  @SideEffectFree public void error(Object message, Throwable t);
+  @SideEffectFree public void fatal(Object message);
+  @SideEffectFree public void fatal(Object message, Throwable t);
+  @SideEffectFree public void log(Priority priority, Object message);
+  @SideEffectFree public void log(Priority priority, Object message, Throwable t);
+  @SideEffectFree public void log(String callerFQCN, Priority level, Object message, Throwable t);
+}
+
+class Logger extends Category {
+  @SideEffectFree public void trace(Object message);
+  @SideEffectFree public void trace(Object message, Throwable t);
+}
+
+// Log4j 2.x API stubs (package org.apache.logging.log4j).
+package org.apache.logging.log4j;
+
+import org.apache.logging.log4j.message.Message;
+
+interface Logger {
+  @SideEffectFree void trace(CharSequence message);
+  @SideEffectFree void trace(CharSequence message, Throwable throwable);
+  @SideEffectFree void trace(Object message);
+  @SideEffectFree void trace(Object message, Throwable throwable);
+  @SideEffectFree void trace(String message);
+  @SideEffectFree void trace(String message, Throwable throwable);
+  @SideEffectFree void trace(String message, Object p0);
+  @SideEffectFree void trace(String message, Object p0, Object p1);
+  @SideEffectFree void trace(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void trace(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void trace(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void trace(String message, Object... params);
+  @SideEffectFree void trace(Message message);
+  @SideEffectFree void trace(Message message, Throwable throwable);
+
+  @SideEffectFree void debug(CharSequence message);
+  @SideEffectFree void debug(CharSequence message, Throwable throwable);
+  @SideEffectFree void debug(Object message);
+  @SideEffectFree void debug(Object message, Throwable throwable);
+  @SideEffectFree void debug(String message);
+  @SideEffectFree void debug(String message, Throwable throwable);
+  @SideEffectFree void debug(String message, Object p0);
+  @SideEffectFree void debug(String message, Object p0, Object p1);
+  @SideEffectFree void debug(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void debug(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void debug(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void debug(String message, Object... params);
+  @SideEffectFree void debug(Message message);
+  @SideEffectFree void debug(Message message, Throwable throwable);
+
+  @SideEffectFree void info(CharSequence message);
+  @SideEffectFree void info(CharSequence message, Throwable throwable);
+  @SideEffectFree void info(Object message);
+  @SideEffectFree void info(Object message, Throwable throwable);
+  @SideEffectFree void info(String message);
+  @SideEffectFree void info(String message, Throwable throwable);
+  @SideEffectFree void info(String message, Object p0);
+  @SideEffectFree void info(String message, Object p0, Object p1);
+  @SideEffectFree void info(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void info(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void info(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void info(String message, Object... params);
+  @SideEffectFree void info(Message message);
+  @SideEffectFree void info(Message message, Throwable throwable);
+
+  @SideEffectFree void warn(CharSequence message);
+  @SideEffectFree void warn(CharSequence message, Throwable throwable);
+  @SideEffectFree void warn(Object message);
+  @SideEffectFree void warn(Object message, Throwable throwable);
+  @SideEffectFree void warn(String message);
+  @SideEffectFree void warn(String message, Throwable throwable);
+  @SideEffectFree void warn(String message, Object p0);
+  @SideEffectFree void warn(String message, Object p0, Object p1);
+  @SideEffectFree void warn(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void warn(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void warn(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void warn(String message, Object... params);
+  @SideEffectFree void warn(Message message);
+  @SideEffectFree void warn(Message message, Throwable throwable);
+
+  @SideEffectFree void error(CharSequence message);
+  @SideEffectFree void error(CharSequence message, Throwable throwable);
+  @SideEffectFree void error(Object message);
+  @SideEffectFree void error(Object message, Throwable throwable);
+  @SideEffectFree void error(String message);
+  @SideEffectFree void error(String message, Throwable throwable);
+  @SideEffectFree void error(String message, Object p0);
+  @SideEffectFree void error(String message, Object p0, Object p1);
+  @SideEffectFree void error(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void error(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void error(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void error(String message, Object... params);
+  @SideEffectFree void error(Message message);
+  @SideEffectFree void error(Message message, Throwable throwable);
+
+  @SideEffectFree void fatal(CharSequence message);
+  @SideEffectFree void fatal(CharSequence message, Throwable throwable);
+  @SideEffectFree void fatal(Object message);
+  @SideEffectFree void fatal(Object message, Throwable throwable);
+  @SideEffectFree void fatal(String message);
+  @SideEffectFree void fatal(String message, Throwable throwable);
+  @SideEffectFree void fatal(String message, Object p0);
+  @SideEffectFree void fatal(String message, Object p0, Object p1);
+  @SideEffectFree void fatal(String message, Object p0, Object p1, Object p2);
+  @SideEffectFree void fatal(String message, Object p0, Object p1, Object p2, Object p3);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8);
+  @SideEffectFree void fatal(
+      String message, Object p0, Object p1, Object p2, Object p3, Object p4, Object p5,
+      Object p6, Object p7, Object p8, Object p9);
+  @SideEffectFree void fatal(String message, Object... params);
+  @SideEffectFree void fatal(Message message);
+  @SideEffectFree void fatal(Message message, Throwable throwable);
+}

checker/tests/resourceleak/TwoResourcesECM.java

@@ -12,19 +12,12 @@
 class TwoResourcesECM {
   @Owning Socket s1, s2;
 
-  // The contracts.postcondition error below is thrown because s1 is not final,
-  // and therefore might theoretically be side-effected by the call to s2.close()
-  // even on the non-exceptional path. See ReplicaInputStreams.java for a variant
-  // of this test where such an error is not issued. Because this method can leak
-  // along both regular and exceptional exits, both errors are issued.
-  //
   // The contracts.exceptional.postcondition error is thrown because destructors
   // have to close their resources even on exception.  If s1.close() throws an
   // exception, then s2.close() will not be called.
   @EnsuresCalledMethods(
       value = {"this.s1", "this.s2"},
       methods = {"close"})
-  // :: error: [contracts.postcondition]
   // :: error: [contracts.exceptional.postcondition]
   public void dispose() throws IOException {
     s1.close();

checker/tests/resourceleak/sideeffect-free-stubs-tests/AutoCloseableClose.java

@@ -0,0 +1,25 @@
+// RLC uses Called Methods facts to remember that a resource has already been closed.
+// This test checks that the RLC-specific SideEffectFree stub for AutoCloseable.close()
+// preserves those facts across another close call in the same destructor, rather than
+// conservatively forgetting them after the first invocation.
+
+import org.checkerframework.checker.calledmethods.qual.EnsuresCalledMethods;
+import org.checkerframework.checker.mustcall.qual.Owning;
+
+class AutoCloseableClose implements AutoCloseable {
+  private @Owning AutoCloseable first = new AutoCloseableResource();
+  private @Owning AutoCloseable second = new AutoCloseableResource();
+
+  @Override
+  @EnsuresCalledMethods(
+      value = {"this.first", "this.second"},
+      methods = "close")
+  public void close() {
+    try {
+      first.close();
+      second.close();
+    } catch (Exception e) {
+      throw new AssertionError(e);
+    }
+  }
+}

checker/tests/resourceleak/sideeffect-free-stubs-tests/CloseableClose.java

@@ -0,0 +1,29 @@
+// RLC uses Called Methods facts to remember that a resource has already been closed.
+// This test checks that the RLC-specific SideEffectFree stub for Closeable.close()
+// preserves those facts across another close call in the same destructor, rather than
+// conservatively forgetting them after the first invocation.
+
+import java.io.Closeable;
+import org.checkerframework.checker.calledmethods.qual.EnsuresCalledMethods;
+import org.checkerframework.checker.mustcall.qual.Owning;
+
+class CloseableClose implements Closeable {
+  private @Owning Closeable first = new CloseableResource();
+  private @Owning Closeable second = new CloseableResource();
+
+  @Override
+  @EnsuresCalledMethods(
+      value = {"this.first", "this.second"},
+      methods = "close")
+  public void close() {
+    try {
+      try {
+        first.close();
+      } catch (Exception ignored) {
+      }
+      second.close();
+    } catch (Exception e) {
+      throw new AssertionError(e);
+    }
+  }
+}

checker/tests/resourceleak/sideeffect-free-stubs-tests/CloseableResource.java

@@ -0,0 +1,14 @@
+// Class representing reources that implements Closeable and Autocloseable to be used by other tests
+// in this directory.
+
+import java.io.Closeable;
+
+final class CloseableResource implements Closeable {
+  @Override
+  public void close() {}
+}
+
+final class AutoCloseableResource implements AutoCloseable {
+  @Override
+  public void close() {}
+}