#include<stdio.h>
int main()
{
	int n,s,i;
	int a[n];
	while(scanf("%d",&n))
	{
		for(s=1,i=0;i<n;i++)
		{
			scanf("%d",a);
			if(a[i] % 2 != 0)
			s = s * a[i];
		}
		printf("%d\n",s);
	}
}

两处错误：
1.int a[n];你的n都没有附上值，怎么能这样申请数组的空间？
改成用malloc动态申请
2.scanf(“%d”,a);要改成scanf(“%d”,&a[i]);

#include<stdio.h>
#include<stdlib.h>
int main()
{
	int n,s,i;

	while(scanf("%d",&n))
	{
		int *a = (int *)malloc(sizeof(int) * n);
		for(s=1,i=0;i<n;i++)
		{
			scanf("%d",&a[i]);
			if(a[i] % 2 != 0)
			s = s * a[i];
		}
		printf("%d\n",s);
		free(a);
	}
}

假如用了C99 ;n 同样必须初始化
另外，输入的时候，每次只是输入a[0] 其他数据并没有输入，却在应用
#include<stdio.h>
void demo(int n){
int n,s,i;
int a[n];//C99 用 变量定义数组（变量作为数组的大小）
for(s=1,i=0;i<n;i++)
{
/// scanf(“%d”,a); //只是输入a[0] 而已，当 i!=0 时候，并没有输入a[i]
scanf(“%d”,a+i); //改成 a+i 或 &a[i] 表示要输入a[i];
//scanf 的实参，需要对象（数据）的地址，而非一个（整型，浮点型，字符型）值。
if(a[i] % 2 != 0)
s = s * a[i];      //这里一直在使用a[i] ，不改的话，除了a[0] 都没有初始化。
}
printf(“%d\n”,s);
}
int main()
{
int n;
while(scanf(“%d”,&n))
{
demo(n);//为了清晰起见，代码都移到函数中了，直接写在这里也可以
}
return 0;
}

C99动态数组，不是先定义一个不明确的大小
而是要定义一个明确的大小
定义以后，数组的大小，就是固定的了
本人举的这个例子里，n 就是每次输入的n 值，假如不是再循环里面
数组的大小是不会随着变化的
例如
C99动态数组中

scanf("%d",n);
int a[n]; //这里数组的元素个数，就是刚才输入的n 的值
.......
scanf("%d",n);//这个新n值，跟数组的元素个数无关，数组保持上次的大小
 while(scanf("%d",&n))
    { ///这里进入新的语句组作用域
          int a[n]; ///C99中，这个数组是每次循环，会重新初始化的，数组元素个数，跟新输入的n 保持一致、
          /// demo(n);//为了清晰起见，代码都移到函数中了，直接写在这里也可以
         n=10;///这里修改了n ，但是数组的元素个数，保持不变
    }

你的代码中，n没有初始化，因此这个数组的元素个数，是不能确定的，也许很大，也许很小
很大的话。就会占用很大内存空间，很小的话，可能数组空间，就不够用了