pic

So, today I want to tell you how you should develop a programming language that will never surrender to a static analyzer. After all, we all know that static analysis is bad, quality control is for those who cannot write clean code, and all unit tests should be written by junior programmers! If you agree with at least one point, write me an e-mail, or better in comments. I must convince myself that such people exist.

Increased focus on software quality is the most effective way to increase programmer productivity.

                      – Code Complete, Steve McConnel

A small disclaimer for those who are against static analysis and code quality control tools.

It is proved mathematically that static analysis can never be 100% true. This is impossible to write a program that understands how another program will execute without first running it.

However, this does not mean that static analysis is useless or that it cannot be effective in finding bugs and vulnerabilities in software. On a slightly more serious note, what makes a programming language difficult to analyze?


Syntax sugar

Let’s take a look at syntax sugar, how it usually looks?

var post = someVariableDeclaredBefore;

Things like var on the one hand help the programmer write code shorter than it could be, and then in Runtime a wise compiler will put in the missing pieces of code, and oh miracle, everything is fine. How does a static analyzer resist this? Here we have one option - to analyze compiled code. In java, for example, these are .class files. This approach has many disadvantages. For example, the first one is that the analysis won’t be smooth as if we were doing it with the source code. The second one is that it’s very difficult or impossible to write plugins for the IDE. You are also unable to analyze comments. But you’re not much bound to the version of the language you’re analyzing, as if you were analyzing the source code. And you don’t have to parse source code files as you often do with third-party tools. So, do you remember this? Now add as much syntax sugar to your language as you can!


Massive AST

This point is very easy to understand, because the bigger your AST is, the harder it is to parse, lexify, etc. Imagine all these giant constructions nested in each other, sometimes it’s just hard to understand what’s written here. But try to analyze it… So you should increase your AST to Statue of Unity size.


Mutability

Well, when I talk about mutability in this context, I mean reassigning the same variable multiple times with parallel reading from it. The difficulty of analysis here lies not only in the equal sign.

File f = new File("foo.txt");
f.doSmth();
f.setPath("bar.txt");
f.write(content);

Changing statements multiple times makes analysis difficult, because keeping track of execution flows becomes a complex task. We must teach our tool to remember when and how each statement changes. If we can’t create setters, the code might look like this:

final File f1 = new File("foo.txt");
final File f2 = new File("bar.txt");
f1.doSmth();
f2.write(content);

Now, for each file we have only one way of a sequence. This piece of code is much easier to analyze, isn’t it? Now you know that you need to ban immutability in your artwork from the world of programming languages.


Multiple conditional flow

How often do you see something like that:

String someCoolMethod(String unsafe) {
  // code above
  if (unsafe.isEmpty()) {
    return safeVariable;
  } else {
    return variableThatDependsOnUnsafeString;
  }
}

Here we see that we can rely on this method only for cases from the first branch. The math here is simple: safe behavior + unsafe behavior = unsafe behavior (1) – it’s called merging. Why is the code I cited above problematic? Let me explain. Imagine we don’t use merging and the method is called multiple times in a row. Let it be called 5 times. How many conditional flows will be produced? If you’re an optimist, you said 10 perhaps. If you know math a little better, you probably said something like: the number of possible branches is the base of the degree, and the number of challenges is the exponent of the degree. And you were right. The answer 2^5 = 32 is conditional flows you have to analyze. But, what if the method you are analyzing has about 10 conditions with only 2 branches? Yes, that’s 1024 condition flows. How does a static analysis tool learn to do this? It’s easier to say we have a vulnerability to keep all possible variations in mind. If we merge each branch into one using the formula (1) with the previous condition and we have 5 method calls, we get 5 conditional flows. I think you can still familiarize yourself with the concept of cyclomatic complexity. So do not forget this when you create your language. Make it impossible for the compiler to compile a program until a method contains at least three, and preferably four, if branches within it.


Polymorphism

What can you say about this quote?

The goal of polymorphism is essentially the opposite of static code analysis.

                                 – Arno Haase

Before I can explain it to you, let’s look at this piece of code.

interface Request {
  Response act();
}

class RqSafe implements Request { 
  @Override
  Response act() {
    return this.doSafe(); // safe implementation
  }
}

class RqUnsafe implements Request { 
  @Override
  Response act() {
    return this.doUnsafe(); // unsafe implementation
  }
}

public List<Response> someProcessingMethod(final Collection<Request> reqs) {
  return reqs.stream()
    .map(r -> r.act()) // boom!
    .collect(Collectors.toList());
}

What should we do here? How can we easily check which implementation of the act() method is safe and which is unsafe? What do we have to do if we can only see contracts from interfaces? Imagine if the library you’re using only contains an open interface, and you don’t know what specific implementation will be used in the code you write? How easy is it to analyze? Polymorphism is about making it hard to figure out what is going on inside, when static analysis is about finding out and knowing exactly what it does when we call a method. This is a big problem for static analysis tools to figure out. Please don’t misunderstand me. I’m not saying that polymorphism is a bad thing. I’m saying that it makes the code more difficult to analyze. How to resist polymorphism? We can use Call Graph. This is just a list of possible targets for each function call statement. We need to look at the act() invocation and figure out which implementations it can go to. We do not know what was placed inside the reqs parameter. Therefore, in our case it will be both implementations, RqSafe and RqUnsafe. Going back to your brand new programming language, given the above, you need to make it impossible to write implementations within a single module.


Conclusion

Here I’ve listed a few things that make static analysis tricky, I’m sure it’s not the whole list, but I found these things quite interesting. As I’ve said before, static analysis can’t be right. So when we find a vulnerability, it’s hard to present the correct results. Here is the dilemma between the two states:

1) Report anything we can find.

2) I’m not sure if I need to report it.

It’s all about trade-off. Thank you; I hope this post was interesting for you, also you can correct me in the comments if I made mistakes, etc.