使用 malloced char 数组时无法创建文件

Question

我正在按照this 文章编写标准 C 中的 PE 文件查看器。它非常整洁，但我对第一行有疑问：

#include "stdafx.h"
#include "Windows.h"
#include <iostream>

int main(int argc, char* argv[]) {
    const int MAX_FILEPATH = 255;
    char fileName[MAX_FILEPATH] = {0};
    memcpy_s(&fileName, MAX_FILEPATH, argv[1], MAX_FILEPATH);
    ...
}

如您所见，作者将 MAX_FILEPATH 定义为 255，但我不能这样做，因为我将给出的输入参数可能比这更大。

我决定使用 malloc 进行动态内存分配，这样我就可以拥有一个自动调整大小的数组（确切的参数长度），并且还可以为我保存 'memcpy_s' 函数：

#include <stdio.h>

#include "Windows.h"

#pragma warning (disable: 6011 6386 6387) // MSVC is giving me a headache.

int main(int argc, char *argv[]) {
    int length = strlen(argv[1]); // Yes, I only care for 1 input argument.
    char* path = malloc(length * sizeof(char));

    int i = 0;
    for (; i <= length; i++) {
        if (i < length) {
            path[i] = argv[1][i];
        }
        else {
            path[i] = '\0';
        }
    }
    ...
}

我知道“for cicle”在速度方面可能不是最佳选择，但我对此并不担心。相反，我很担心，因为一切似乎都很好，但是当我到达代码的第二部分（即 HANDLE 的创建）时，它总是返回错误。

HANDLE file = CreateFileA(path, GENERIC_ALL, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);

if (file == INVALID_HANDLE_VALUE) {
    printf_s("Nope."); // Always throwing nopes.
}

你知道我做错了什么吗？也许我试图混合不同的数据类型或函数的方式......？

非常感谢，提前。

顺便说一句，如果您对我的 C 编码风格有任何建议，或者如果某些东西不是真正的“C 标准”，请告诉我。

如果有人遇到同样的问题...

尝试以管理员身份运行您的程序...这完全与文件权限有关。

Answer 1

除非有其他原因，否则无需复制argv[1]，您可以直接使用它：

#include <stdio.h>
#include <stdlib.h>
#include <Windows.h> // use normal include guards

int main(int argc, char *argv[])
{
    if (argc < 2){ // if the argument is not there the program would have undefined behavior
        fprintf(stderr, "Usage: <prog> <filename>\n");
        return EXIT_FAILURE;
    }
    
    HANDLE file = CreateFileA(argv[1], GENERIC_ALL, FILE_SHARE_READ, NULL, OPEN_EXISTING, FILE_ATTRIBUTE_NORMAL, NULL);
    //                        ^^^^^^^
    if (file == INVALID_HANDLE_VALUE) {
        printf_s("Nope."); // Always throwing nopes.
    }
    //...
}

在任何情况下，您都应该检查argv[1] 是否真的存在，否则您的程序将出现未定义的行为。

无论如何，如果你真的想复制，还有更好的选择，这里是其中之一：

#include <stdio.h>
#include <stdlib.h>
#include<string.h> // maybe needed for strdup
#include <Windows.h> // use normal include guards


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

    if(argc < 2){ // if the argument is not there the program would have undefined behavior
        fprintf(stderr, "Usage: <prog> <filename>\n");
        return EXIT_FAILURE;
    }
    // allocates memory for the new string and duplicates it
    char* path = _strdup(argv[1]); // MSVC can complain about POSIX strdup()
    //...
    // you'll need to free path
}

使用正确的数组大小，您仍然可以使用memcpy 或strcpy 库函数之一或其变体之一：

int length = strlen(argv[1]) + 1; // account for null byte
char *path = malloc(length); // sizeof char is always 1 byte
memcpy_s(path, length, argv[1], length); // path is already pointer, removing &

请注意，在您的代码中，filename 在作为参数传递时会衰减为指向其第一个元素的指针，因此您还应该删除 &。

或者干脆用指针：

char *path = argv[1];

虽然指针选项与直接使用argv[1]基本相同，如第一段代码所示。

如果还有与字符串无关的问题，GetLastError() 将帮助您查明问题所在。