-1

この問題があります。構造体を埋める関数 create_message があります

struct message {
struct header{
    char *protocol_version;         
    char *type;                     
    long int sequence_number;       
}header;                            
struct body
{
    int num_tag;                    
    char *tag_labels[LEN];          
    int num_attr_tag[LEN];          
    char *attr_labels[LEN][LEN];
    char *attr_values[LEN][LEN];    
    char *attr_types[LEN][LEN];         }body;                              
  }; 

create_message のコードは次のとおりです。

void create_message(struct message *msg, CharmsMsg *chmsg, char *protocol_version)
{
int i,j,n;
XTypes type = chmsg->type;
printf("Tipo %ld\n", chmsg->type);

msg->header.protocol_version = (char *) malloc(sizeof(char)*STRLEN);
msg->header.protocol_version = protocol_version;

msg->header.type = (char *) malloc(sizeof(char)*STRLEN);
msg->header.type = metadati[type].typemsg;

msg->header.sequence_number = chmsg->SeqNo;

msg->body.num_tag = metadati[type].num_tag;

n=0;
for(i=0;i<msg->body.num_tag;i++)        
{
    // assegno le etichette ai tag
    msg->body.tag_labels[i] = (char *) malloc(sizeof(char)*STRLEN);
    msg->body.tag_labels[i] = metadati[type].tag_labels[i];
    msg->body.num_attr_tag[i] = metadati[type].num_attr_tag[i];

    for(j=0;j<msg->body.num_attr_tag[i];j++)     
    {

        msg->body.attr_labels[i][j] = (char *) malloc(sizeof(char)*STRLEN);
        msg->body.attr_labels[i][j] = metadati[type].attr_labels[i][j];

        // assegno il valore dell'attributo j-esimo al tag i-esimo
        msg->body.attr_values[i][j] = (char *) malloc(sizeof(char)*STRLEN);
        msg->body.attr_values[i][j] = ((getDati[type])(chmsg->structCHARMS))[n++];
        printf("IN create_message: Contenuto buf[%d] %s\n",(i+j), msg->body.attr_values[i][j]);
        // assegno il tipo dell'attributo j-esimo al tag i-esimo
        msg->body.attr_types[i][j] = (char *) malloc(sizeof(char)*STRLEN);
        msg->body.attr_types[i][j] = metadati[type].attr_types[i][j];
    }
}
 }

特に create_message は、関数へのポインタの配列である getDati[type] を呼び出します。この配列内の特定の関数は、以下に示す getRegisterMe です。

char **getRegisterMe(void *structCHARMS)
{
ClientData *client = (ClientData *)structCHARMS;
char *buf[3];
int i;

for(i=0; i< 3; i++)
    buf[i] = (char*)calloc(MAX_SEQ_LEN, sizeof(char));

//attributi del tag[0]:Cookie
sprintf(buf[0], "%d", client->cookie_value);
//attributi del tag[1]:Register
sprintf(buf[1], "%s", addrtostr(client->local_addr));
sprintf(buf[2], "%d", client->mode);

printf("IN getData buf[0]%s\n",buf[0]);
printf("IN getData buf[1]%s\n",buf[1]);
printf("IN getData buf[2]%s\n",buf[2]);

return (char **)buf;
}

プログラムを起動すると、getRegisterMe が struct struct メッセージの要素を正しく埋めません。実際、私はこれを観察します:

IN getData buf[0]0
IN getData buf[1]127.0.0.1:53285
IN getData buf[2]3
IN create_message: Contenuto buf[0] ?   ?V׾??FR?
IN getData buf[0]0
IN getData buf[1]127.0.0.1:53285
IN getData buf[2]3
IN create_message: Contenuto buf[1] H??H???H??t?

IN getData buf[0]0
IN getData buf[1]127.0.0.1:53285
IN getData buf[2]3
IN create_message: Contenuto buf[2] 

getRegisterMe で出力される char*buf の値が、msg->body.attr_values で出力される値と一致しないのはなぜですか?? 誰でも私を助けることができますか?

4

2 に答える 2

1

関数 getRegisterMe は、ローカル変数 buf へのポインターを返します。このポインターは、関数が戻ると消えます。

于 2013-09-20T16:07:17.783 に答える
1

コードには、メモリを割り当ててポインターに割り当てる場所がいくつかあります。次に、次の行で、ポインターが別の場所を指しているため、(a) 割り当てたメモリをリークし、(b) 所有していない渡されたデータにポインターをリダイレクトしました。 .

msg->header.protocol_version = (char *) malloc(sizeof(char)*STRLEN);
msg->header.protocol_version = protocol_version;

代わりにデータをコピーします。

msg->header.protocol_version = (char *) malloc(sizeof(char)*STRLEN);
strncpy(msg->header.protocol_version, protocol_version, STRLEN);
于 2013-09-20T16:09:23.857 に答える