libmysql：警告：返回的局部变量“行”的地址（C++/C）

Question

我正在使用 g++。

为了方便使用 libmysql，实现了一些包装器。还制作了与 C/C++ 兼容的代码。

static MYSQL_RES *db_query(MYSQL *db, const char *query, va_list params) {
    char q[500];
    vsprintf(q, query, params);
    va_end(params);
    if (mysql_query(db, q)) {
        fprintf(stderr, "%s\n", mysql_error(db));
        return NULL;
    }

    MYSQL_RES *res = mysql_use_result(db);
    return res;
}

MYSQL_ROW* db_query_all(MYSQL *db, const char* query, ...) {
        va_list params;
    MYSQL_RES *res = db_query(db, query, params);

    int count = mysql_num_rows(res);

    MYSQL_ROW rows[count];
    for (int i = 0; i<count; i++) 
    {
        rows[i] = mysql_fetch_row(res);
    }

    mysql_free_result(res);
    return rows;
}

并在编译期间收到警告：

warning: address of local variable ‘rows’ returned [-Wreturn-local-addr]
     MYSQL_ROW rows[count];

请帮我找出问题所在。我无法搜索与我的特定案例相关的内容。

Answer 1

一种更 C++ 的方法是定义一个 movable 缓冲区类型：

struct Buf {
  Buf(size_t size) : mPtr(std::make_unique<MYSQL_ROW[]>(size)), mSize(size) {}

  std::unique_ptr<MYSQL_ROW[]> mPtr;
  size_t mSize;
};

然后你可以这样实现函数：

Buf db_query_all(MYSQL *db, const char* query, ...) {
        va_list params;
    MYSQL_RES *res = db_query(db, query, params);

    int count = mysql_num_rows(res);

    Buf rows(count);
    for (int i = 0; i<count; i++) 
    {
        rows.mPtr[i] = mysql_fetch_row(res);
    }

    mysql_free_result(res);
    return std::move(rows);
}

这种方法的最大好处是您不必手动处理内存。

Answer 2

rows 是 db_query_all 函数的本地数组，一旦控制退出函数就会被销毁，因此返回它的引用是未定义的行为。

您可以动态分配内存并返回对它的引用。

MYSQL_ROW* db_query_all(MYSQL *db, const char* query, ...) {
        va_list params;
    MYSQL_RES *res = db_query(db, query, params);

    int count = mysql_num_rows(res);

    MYSQL_ROW *rows = malloc(sizeof(*rows)*count);
    for (int i = 0; i<count; i++) 
    {
        rows[i] = mysql_fetch_row(res);
    }

    mysql_free_result(res);
    return rows;
}

请注意，一旦完成处理，您需要free 内存。

Answer 3

您的错误信息来自这一行：

MYSQL_ROW rows[count];

这意味着rows 将被分配到堆栈上，这意味着一旦你的函数退出，你的变量的内容将是未定义的。

你需要：

• 在堆上分配变量（推荐）

MYSQL_ROW rows[] = calloc(count,sizeof(MYSQL_ROW));

一旦你不再需要结果，别忘了free

• 从上游传递行作为参数
  这将需要您在另一个步骤中获得计数