为什么在glibc中使用crypt会引起编译器警告?

时间:2019-01-06 04:13:34

标签: c clang compiler-warnings glibc crypt

我尝试编译以下代码(最小示例,请参见整个代码的编辑内容):

// a.c
#include <stdio.h>

#define _XOPEN_SOURCE
#include <unistd.h>

int main(int argc, char* argv[])
{
    puts((const char*) crypt("AAAA", "$6$2222"));
    return 0;
}

使用clang-7 -lcrypt a.c并发出以下警告:

minimum.c:8:24: warning: implicit declaration of function 'crypt' is invalid in C99 [-Wimplicit-function-declaration]
    puts((const char*) crypt("AAAA", "$6$2222"));
                       ^
minimum.c:8:10: warning: cast to 'const char *' from smaller integer type 'int' [-Wint-to-pointer-cast]
    puts((const char*) crypt("AAAA", "$6$2222"));
         ^
2 warnings generated.

但是./a.out确实有效:

$6$2222$6GKY4KPtBqD9jAhwxIZGDqEShaBaw.pkyJxjvSlKmtygDXKQ2Q62CPY98MPIZbz2h6iMCgLTVEYplzp.naYLz1

我发现,如果我这样删除#include <stdio.h>puts

// new_a.c
#define _XOPEN_SOURCE
#include <unistd.h>

int main(int argc, char* argv[])
{
    crypt("AAAA", "$6$2222");
    return 0;
}

然后没有警告。

如何在不删除#include <stdio.h>的情况下解决这些警告?

编辑:

整个程序:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define _X_OPEN_SOURCE
#include <unistd.h>

#include <assert.h>

void* Calloc(size_t cnt, size_t size)
{
    void *ret = calloc(cnt, size);
    assert(ret);
    return ret;
}

size_t GetSaltLen(const char *salt)
{
    size_t salt_len = strlen(salt);
    assert(salt_len >  0);
    assert(salt_len <= 16);
    return salt_len;
}

char* GetSaltAndVersion(const char version, const char *salt)
{
    size_t saltlen = GetSaltLen(salt);
    /*
     * The format of salt:
     *     $one_digit_number$up_to_16_character\0
     * For more info, check man crypt.
     */
    char *ret = (char*) Calloc(1 + 1 + 1 + saltlen + 1, sizeof(char));

    char *beg = ret;

    *beg++ = '$';
    *beg++ = version;
    *beg++ = '$';
    memcpy((void*) beg, (const void*) salt, saltlen + 1);

    return ret;
}

void crypt_and_print(const char *passwd, const char *salt_and_version)
{
    char *result = crypt(passwd, salt_and_version);
    assert(puts(result) != EOF);
}

int main(int argc, char* argv[])
{
    if (argc != 4) {
        fprintf(stderr, "argc = %d\n", argc);
        return 1;
    }

    char *salt_and_version = GetSaltAndVersion(argv[2][0], argv[3]);
    crypt_and_print(argv[1], salt_and_version);
    free(salt_and_version);

    return 0;
}

我已经按照@Andrey Akhmetov的建议进行了尝试,并将#define放在第一行,但是警告并没有消失。

1 个答案:

答案 0 :(得分:5)

feature_test_macros(7)中记录了宏_XOPEN_SOURCE。联机帮助页特别指出:

  

注意:为了有效,必须在包含任何头文件之前定义功能测试宏 。可以在编译命令(cc -DMACRO=value中完成此操作,也可以在包含任何标头之前在源代码中定义宏。

当包含stdio.h时,将间接包含features.h,该{@ 1}使用此时在定义的功能测试宏。特别是,由于此时尚未定义_XOPEN_SOURCE和朋友,因此crypt.h不会声明crypt

在您定义_XOPEN_SOURCE时,为时已晚,因为features.h具有包含保护,可防止将其两次包含。

通过交换前两行的顺序,代码可以正常工作,而不会在我的系统上发出此警告:

#define _XOPEN_SOURCE
#include <stdio.h>
#include <unistd.h>

int main(int argc, char* argv[])
{
    puts((const char*) crypt("AAAA", "$6$2222"));
    return 0;
}

您的较大示例不起作用还有第二个原因:您将_X_OPEN_SOURCE写为宏的名称,而正确的名称是_XOPEN_SOURCE