You check mlist to see if it's null then log this. If it's null, you have no guard around the size because you just log that there's a null entry and continue processing as per normal. Try something like this.
if (mlist == null) {
log.error("....");
return 0;
}
return mlist.size();