各位大侠，求指教！！！

char* pick_4(char* s1,char *s2)                //剔除s1中的4，并送到s2
{
        char *s;
        s = s2 ;
        while(*s1 != ''){
                if( *s1 != '4')
                        *s2++ = *s1;
                s1++;
                }
        *s2 = '';
       
        return s;
}


输入的s1 = "123456",s2也是有定义的，只不过是个空指针，但是运行不了。
请问这个函数有什么问题么？？
多谢各位大侠！