1

Linux用のモジュールを書いているときに文字列を操作するのに苦労しています。私の問題は、異なる値を持つintArray[10]があることです。my_readプロシージャでバッファに送信できるように、文字列を生成する必要があります。配列が{0,1,112,20,4,0,0,0,0,0}の場合、出力は次のようになります。

0:(0)
1:-(1)
2:-------------------------------------------------------------------------------------------------------(112)
3:--------------------(20)
4:----(4)
5:(0)
6:(0)
7:(0)
8:(0)
9:(0)

上記の文字列をchar[]配列に配置しようとすると、奇妙な文字がそこに表示されてしまいます

これがコードです

    int my_read (char *page, char **start, off_t off, int count, int *eof, void *data)
{
 int len;   
 if (off > 0){
  *eof =1;
  return 0;
 }
     /* get process tree */
 int task_dep=0;   /* depth of a task from INIT*/
 get_task_tree(&init_task,task_dep);

 char tmp[1024];

 char A[ProcPerDepth[0]],B[ProcPerDepth[1]],C[ProcPerDepth[2]],D[ProcPerDepth[3]],E[ProcPerDepth[4]],F[ProcPerDepth[5]],G[ProcPerDepth[6]],H[ProcPerDepth[7]],I[ProcPerDepth[8]],J[ProcPerDepth[9]];
 int i=0;
 for (i=0;i<1024;i++){ tmp[i]='\0';}

 memset(A, '\0', sizeof(A));memset(B, '\0', sizeof(B));memset(C, '\0', sizeof(C));
 memset(D, '\0', sizeof(D));memset(E, '\0', sizeof(E));memset(F, '\0', sizeof(F));
 memset(G, '\0', sizeof(G));memset(H, '\0', sizeof(H));memset(I, '\0', sizeof(I));memset(J, '\0', sizeof(J));
 printk("A:%s\nB:%s\nC:%s\nD:%s\nE:%s\nF:%s\nG:%s\nH:%s\nI:%s\nJ:%s\n",A,B,C,D,E,F,G,H,I,J);
 memset(A,'-',sizeof(A));
 memset(B,'-',sizeof(B));
 memset(C,'-',sizeof(C));
 memset(D,'-',sizeof(D));
 memset(E,'-',sizeof(E));
 memset(F,'-',sizeof(F));
 memset(G,'-',sizeof(G));
 memset(H,'-',sizeof(H));
 memset(I,'-',sizeof(I));
 memset(J,'-',sizeof(J));
 printk("A:%s\nB:%s\nC:%s\nD:%s\nE:%s\nF:%s\nG:%s\nH:%s\nI:%s\nJ:%\n",A,B,C,D,E,F,G,H,I,J);


 len = sprintf(page,"0:%s(%d)\n1:%s(%d)\n2:%s(%d)\n3:%s(%d)\n4:%s(%d)\n5:%s(%d)\n6:%s(%d)\n7:%s(%d)\n8:%s(%d)\n9:%s(%d)\n",A,ProcPerDepth[0],B,ProcPerDepth[1],C,ProcPerDepth[2],D,ProcPerDepth[3],E,ProcPerDepth[4],F,ProcPerDepth[5],G,ProcPerDepth[6],H,ProcPerDepth[7],I,ProcPerDepth[8],J,ProcPerDepth[9]);  

 return len;
}

それはこれでうまくいきました：

 char s[500];
 memset(s,'-',498); 

 for (i=len=0;i<10;++i){ 
      len+=sprintf(page+len,"%d:%.*s(%d)\n",i,ProcPerDepth[i],s,ProcPerDepth[i]); 
}

sprintfに文字列charを掛ける簡単なフラグがあるのだろうか。ありがとう– </ p>

4

1 に答える 1

2

ここにいくつかの問題があります：

  1. A、B、C...配列を文字で完全に埋めました。次に、それらをnullで終了する文字列を予期しているI/Oルーチンに渡します。文字列はnullで終了していないため、printk()運が良ければnullが見つかるまで、オブジェクトの後にスタックメモリにあるものをすべて出力し続けます。

  2. Linuxのようなマルチスレッドカーネルには、スタック割り当てに関して厳密で比較的小さな制約があります。カーネル呼び出しチェーン内のすべてのインスタンスは、特定のサイズに収まる必要があります。収まらない場合、何かが上書きされます。このエラーは検出されない場合があります。メモリの破損がパニックまたはウェッジにつながるため、ある種のダウンストリームクラッシュが発生します。カーネルスタックに大規模で可変の配列を割り当てることは、良い考えではありません。

  3. tmp []配列を記述し、それを適切にnul-terminateする場合は、それを初期化する理由もありません。ただし、初期化する場合は、コンパイラーで生成されたコードを使用して、次のように言うだけで済みますchar tmp[1024] = { 0 }; （アグリゲートの部分的な初期化には、アグリゲート全体のC99初期化が必要です）。他のアレイにも同様の観察が当てはまります。

  4. それらの配列のほとんどとそのコードのほとんどを取り除き、次の行に沿って何かをするのはどうですか？

for(i = j = 0; i < n; ++i)
    j += sprintf(page + j, "...", ...)
于 2011-01-15T22:12:33.353 に答える