fix bug null pointer