某社に作らせたコード

        try {
            inetAddressList[i] = new InternetAddress(toAddressList[i]);
        } catch (AddressException e) {
            // これはしょうがない
        }

おいおい、なにが「しょうがない」んだよ。
不正な書式のアドレスが混入してたら、配列の一部の要素が null になるんだけど。それでもこの後の部分がちゃんと動くとでも思ってるのかなぁ。。。