关于List<E>.addAll(Collection<? extends E> collection)传入参数null

今天关于一个漏洞查了很久，原来是因为List<E>.addAll(Collection<? extends E> collection)传入参数为null产生的。下面就是我解决问题的过程

bug现象是activity有时会出现闪退，于是我查看了logcat，截图如下：

黑色箭头标注处是关键，一般NullPointerException都是由于空指针（null）或者没有初始化产生的，那我再看看下面ArrayList.addAll()，应该是说这个调用这个函数导致了NullPointerException异常的产生，难道是因为ArrayList.addAll()不能传入空指针,函数没有对空指针的情况进行处理，于是我查看了ArrayList.addAll()源码，如下：

由上可知，在ArrayList.addAll()中对传进来的参数没有做null判断，于是，在调用collection.toArray()函数的时候就抛异常了，activity就崩溃了

在使用ArrayList.addAll()的时候一定要注意传入的参数会不会出现为null的情况，如果有，那么我们可以做以下判断

if (collection!= null)
mInfoList.addAll(Collection<? extends E> collection);

如果为null，就不执行下面的了，作为程序员，我们也不能确保是不是存在null的情况，所以为了确保不会出错，在前面加个判断是一个有经验的程序员该做的。