C++でmysql接続を閉じる際にセグメンテーションエラーが発生しました。解決する方法？

Question

libmysql++ APIを使用してデータベースを照会しようとしていますが、これまでにというクエリを正常に実行することができました。 （その理由は、の後ろにいくつかのについては後述する）

だから、すべてが正常に動作しますが、あなたは以下を参照することができ次第、私は接続を閉じてプログラムを終了する試みとして、 Segmentation fault (core dumped)は、表示され

：

shell> g++ -o test testCDB.cpp -L/usr/include/mysql -lmysqlclient -I/usr/include/mysql 

testCDB.cpp: In function ‘int main(int, char**)’: 
testCDB.cpp:122:74: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings] 
    StudentManager manager1("localhost", "root", "ikilledkennedy.", "testDB"); 
                     ^
testCDB.cpp:122:74: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings] 
testCDB.cpp:122:74: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings] 
testCDB.cpp:122:74: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings] 

shell> ./test 
Successfully connected to the database. 

Enter your query: SHOW DATABASES; 

[information_schema] 
[mysql] 
[performance_schema] 
[sys] 
[testDB] 

Do you wish to continue?(Y:1/N:0) : 1 

Enter your query: SHOW TABLES; 

[Student] 

Do you wish to continue?(Y:1/N:0) : 0 
Segmentation fault (core dumped) 

shell> 

この問題の原因は何ですか？これを解決するにはどのような対策が必要ですか？ 一部クエリに部分来る

- Iは、これらのクエリは、結果セットのときIを産生しないためであると感じているすぎるSegmentation faultと出口データベース（等INSERT）に供給される特定のクエリが存在しますdisplayTable()に行を取得しようとしたか、おそらくperformQuery(char *)に結果セットを解放してください。NULLの値が検出されました。私はこれを考えているのでしょうか、あるいは他の根本的な原因がありますか？次のように

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

#include <iostream> 
#include <string> 
#include <mysql.h> 
#include <limits> 

using namespace std; 

class StudentManager 
{ 

    private: 

     char *host; 
     char *user; 
     char *password; 
     char *database; 

     MYSQL *connection; 
     MYSQL_RES *result; 

     void init() 
     { 
      connection = mysql_init(NULL); 

      if(!mysql_real_connect(connection, host, user, password, database, 0, NULL, 0)) 
      { 
       cout << "Cannot connect to the database.\n" 
        << "Please make sure you have entered the details correctly and try again." 
        << endl; 
      } 

      else 
      { 
       cout << "Successfully connected to the database.\n" 
        << endl; 
      }  
     } 

     void displayTable() 
     { 
      MYSQL_ROW row; 

      unsigned int num_fields; 
      unsigned int i; 

      num_fields = mysql_num_fields(result); 

      cout << endl; 

      while((row = mysql_fetch_row(result))) 
      { 
       for(i = 0; i < num_fields; i++) 
       { 
        cout << "[" << (row[i] ? row[i] : "NULL") << "]"; 
       } 

       cout << endl; 
      } 

      cout << endl; 
     } 


    public: 

     StudentManager(char *host, char *user, char *password, char *database) 
     { 
      this->host  = host; 
      this->user  = user; 
      this->password = password; 
      this->database = database; 

      init(); 
     } 

     void performQuery(char *query) 
     { 
      if(mysql_query(connection, query)) 
      { 
       cout << "Query error: " 
        << mysql_error(connection) 
        << endl; 

       return; 
      } 

      result = mysql_use_result(connection); 

      displayTable(); 

      mysql_free_result(result); 
     } 


     void disconnect() 
     { 
      cout << "Closing connection..."; 

      mysql_close(connection); 

      cout << "Connection successfully closed."; 
     } 
}; 


int main(int argc, char *argv[]) 
{ 

    StudentManager manager1("localhost", "foo", "bar", "testDB"); 

    int choice = 1; 

    char *query; 

    while(1) 
    { 
     cout << "Enter your query: "; 
     cin.getline(query, 256); 

     manager1.performQuery(query); 

     cout << "Do you wish to continue?(Y:1/N:0) : "; 
     cin >> choice; 

     cin.ignore(numeric_limits<streamsize>::max(), '\n'); 

     if(choice < 1) 
     { 
      manager1.disconnect(); 

      break; 
     } 
    } 

    return 0; 
} 

Answer 1

あなたが初期化されていないここにポインタを使用します。

char *query; 

while(1) 
{ 
    cout << "Enter your query: "; 
    cin.getline(query, 256); 

query変数が初期化されていない、といくつかのランダムなメモリ位置にポイントを。おそらくスタックセグメントのどこかにあります。 getlineを使用してその場所に書き込むと、スタックが破損します。これはセグメンテーション・フォルトにつながります。

問題のコードは、パフォーマンス重要であるとは思えないので、私はこのようなコードを書き換えることをお勧めしたい：

std::string line; 
while (cin.good()) 
{ 
    cout << "Enter your query: "; 
    std::getline(cin, line); 

PSが私の意見では、コードと他のいくつかの問題がありますが、以来、質問はセグメンテーションの失敗に関するものです。

Answer 2

ここでは、早期に破損したメモリのようになります。

char *query; // <-- uninitialized buffer 

while(1) 
{ 
    cout << "Enter your query: "; 
    cin.getline(query, 256); 

getlineはストリームバッファに格納するからデータを読み出します。適切な長さのバッファは、あなたが割り当てなければなりません。それを修正するには、使用します。

char query[256]; 

をこれらのクエリが結果を生成しないので、私はあると感じ（INSERTなど）のデータベースに送らすぎセグメンテーションフォールトと出口が

を設定し、特定のクエリがあります

INSERTクエリは結果セットを生成しませんので、ドキュメントに従ってmysql_fetch_row(result)はNULLを返します。これは正しい動作であり、理由はSegmentation faultではありません。