0

我正在制作一个模块,在其中我从用户那里获取一些条目并将它们分解成更小的字符串并将它们存储到一个数组中。逻辑工作正常,但几分钟后内核恐慌。下面我给出了将条目从用户空间放入 /proc 文件的代码,然后是模块中的编码。

FILE *fp;
fp =fopen("/proc/tx_info","w");

fprintf(fp,"protocol   address      tos   payload interface");
fprintf(fp,"\n %3s %6s %7d %5d %6s\n",prot,addr,tos,pld,inter);
fclose(fp);

现在是模块中的代码

void tx_break(void)
{
char ch;
char tmp[25];
int i =0;
int k=0,j=0,y = 0;

ch = tx_buffer[0];
//runing the while loop while the ch goes to next line from where the real user entries starts
while(ch != '\n')
     {
       ch = tx_buffer[i];
       i++;
     }


while(ch != '\0')
{
   j=0;
//while loop for blanks in between the user entries
   while(ch == ' ' && ch != '\0')
    {
      i++;
      ch = tx_buffer[i];
    }
//while loop to put the user defined entries to a temporary tmp which would become an entry of 2d array(info.arr[][])
  while(ch != ' ' && ch !='\0')
    {
       tmp[j] = ch;
       j++;
       i++;
       ch = tx_buffer[i];
    }
    tmp[j] = '\0';
    printk(KERN_ALERT"\n%s\n",tmp);
//putting user entries to an 2d array
    for(k=0;tmp[k] != '\0';k++)
         info.arr[y][k] = tmp[k];
         info.arr[y][k] = '\0';
    printk("the 2d::%s",&info.arr[y][0]);
    y++;
   // memcpy(&info.arr[k][0],tmp,j+1);
   // k++;
}
 }
4

1 回答 1

0

我真的不知道您是如何阅读的tx_bufferp,所以我想您已将文件中的所有信息都存储在上面/proc/tx_info

从这里,我看到了一些错误:

while(ch != '\n')
{
    ch = tx_buffer[i];
    i++;
}

在上次执行时保持 ch = '\n'。

现在,使用 ch = '\n', tmp[0] = "\n",您将其存储info.arr[0]并打印其地址。因此,您已将一项非项目存储在info.arr. 这个变量的维数是多少?也许您存储超出范围。

在最后一项处理中,您也将\n字符存储在tmp因为while(ch != ' ' && ch !='\0'). 这个对吗?

于 2012-04-09T10:26:52.090 回答