There is a small error in code changes for 6948909:

     1436 +                    if (i < len && bs[i+1] == '\n') i++;

Obviously, "i < len" cannot guarantee the existence of bs[i+1], it should be "i < len - 1".

This is not a serious bug because we've never supported "\r" as a line separator in manifest although it's specified as a newline in the doc.