for(;*a=='*';a++);p=a;这段代码的意思是获取第一个不是*的字符的地址,作为后面迭代的起始点。这里存在3个问题：
1、循环没有判断结束符号，如果传进来的是长度0的字符串或者是没有*号的字符串就会死循环，所以应该改成for(;*a=='*' && *a!=0;a++);
2、p在这里记录了起始点，实际后面你想要打印输出的就是这个起始点开始的字符串，而不是s，所以这个函数需要一个额外的指针来记录这个起始地址并在函数结束返回出去（你的a和p在后面循环中都改变值了，不能作为字符串起始地址返回）。
3、后面for(;*a!=0;a++)这段代码是从第二个非*字符开始算，而上面循环结束a是指向第一个非*字符，所以上面需要写成p=a++；
最后在main函数中需要打印返回值字符串。
下面是我根据你代码修改的：
#include <stdio.h>
char *fun(char *a)
{
char *p=NULL,*p0=NULL;
for(;*a=='*' && *a!=0;a++);
if(a==0) return NULL;
p0=p=a++;
for(;*a!=0;a++)
if(*a!='*')
*(++p)=*a;
*(++p)=0;
return p0;
}
int main()
{
char s[20]="***AB*Hhk**",*p=NULL;
p=fun(s);
if(p) puts(p);
return 0;
}
/*提议：1、指针在定义的时候请给初值NULL，避免野指针。2、结束符号可以直接用0来赋值不需要用'\0'。3、for或if下面只有一个语句的时候不需要括号，注意语句作用域。
for(;*a!=0;a++)//这里没有大括号，只对下面if起作用
if(*a!='*'){//这里有大括号，只对括号里的语句起作用（既然只有1句，括号不需要）
*(++p)=*a;
}
*(++p)=0;//所以这句是在for循环之外，不应该退行（虽然不会报错，但影响阅读）。*/
