Learn C The Hard WayでCスキルを磨いています。現在、17回目の演習を行っています。
私は「エクストラクレジット」の部分をやっています。そのページに記載されているデータベースコードを作成して、
「MAX_DATAとMAX_ROWSのパラメーターを受け入れるようにコードを変更し、それらをDatabase構造体に格納し、それをファイルに書き込んで、任意のサイズのデータベースを作成します。」
そこで、#defineディレクティブをコメントアウトし、AddressとDatabaseの構造体を次のように変更しました。
struct Address {
int id;
int set;
char *name;
char *email;
};
struct Database {
int MAX_DATA;
int MAX_ROWS;
struct Address *rows;
};
そのため、ユーザーからMAX_DATAパラメーターとMAX_ROWSパラメーターを取得して、ユーザーの好みに応じてデータベースを作成できます。私がコードで変更した他のものは-
Database_create関数:
void Database_create(struct Connection *conn, int MAX_DATA, int MAX_ROWS)
{
int i = 0;
conn->db->MAX_DATA = MAX_DATA;
conn->db->MAX_ROWS = MAX_ROWS;
conn->db->rows = malloc(sizeof(struct Address) * MAX_ROWS);
for(i = 0; i < MAX_ROWS; i++) {
struct Address addr = {.id = i, .set = 0};
conn->db->rows[i] = addr;
}
}
「4バイトの無効な読み取り」を取得している部分で、SegFaultは次のとおりです。
void Database_set(struct Connection *conn, int id, const char *name, const char *email)
{
struct Address *addr = &conn->db->rows[id];
int MAX_DATA = conn->db->MAX_DATA;
if(addr->set) die("Already set, delete it first");
/* This if statement gives error for addr->set */
addr->set = 1;
addr->name = malloc(sizeof(char) * MAX_DATA);
addr->email = malloc(sizeof(char) * MAX_DATA);
// WARNING: bug, read the "How To Break It" and fix this
char *res = strncpy(addr->name, name, MAX_DATA);
// demonstrate the strncpy bug
if(!res) die("Name copy failed");
res = strncpy(addr->email, email, MAX_DATA);
if(!res) die("Email copy failed");
}
このコードスニペットがすべてではないことは知っていますが、ここにコード全体を貼り付けることはできません。だから、私はここにそれを投稿しました:http: //pastebin.com/EbKShT3r'c 'オプションを使用して最初の実行のためのデータベースを作成して書くことができます。ただし、「s」オプションを使用してエントリを追加すると、セグメンテーション違反が発生します。
編集:それで、最後に、@ WhizCraigによって与えられたソリューションでこのプログラムを動作させました。しかし、メモリを解放するために、これが私が試していることです:
void Database_close(struct Connection *conn)
{
int i;
if (conn) {
int MAX_ROWS = conn->db->MAX_ROWS;
for (i=0; i<MAX_ROWS; i++) {
struct Address *row = conn->db->rows+i;
if (row->set) {
free(row->name);
free(row->email);
}
}
free(conn->db->rows);
if(conn->file) fclose(conn->file);
if(conn->db) free(conn->db);
free(conn);
}
}
そして、Valgrindでエラーが発生し、メモリがリークしています。上記のコードのエラーがわかりません。ただし、主な問題は解決されたようです:)