使用umask与chmod系统调用异常：文件权限未按预期修改

看起来你的代码逻辑本身是正确的，但出现权限未按预期设置的情况，最可能的原因有两个，我们逐一拆解：

1. 文件系统不支持Unix-style权限位 #

这是最常见的导致权限设置无效的原因。如果你的1.txt所在的分区是FAT32、NTFS这类非Unix文件系统（比如挂载的Windows分区），这些文件系统无法存储Unix的权限属性，因此无论你用open还是chmod设置什么权限，文件都会显示为rwxrwxrwx，所有权限操作都会被系统忽略。

验证方法：
在系统默认的支持权限的分区（比如/tmp目录）下测试：

./umask /tmp/1.txt 0744
ls -l /tmp/1.txt

如果此时权限显示为-rwxr--r--，那说明就是文件系统的问题。

2. 类型转换的潜在代码缺陷 #

你的代码中用strtol（转换为有符号长整型）来处理无符号的mode_t类型，虽然对于0744这种小数值不会直接出问题，但这不是最佳实践，可能在某些极端场景下导致符号扩展问题。应该使用strtoul来转换为无符号长整型，更匹配mode_t的无符号属性，同时还能添加转换有效性的检查。

修复后的代码 #

针对上述问题，我们改进代码，添加转换错误检查，并使用更合适的类型处理：

#define _XOPEN_SOURCE 700
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <errno.h>
#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define check_error(expr, userMsg) \
	do { \
		if (!(expr)) { \
			perror(userMsg); \
			exit(EXIT_FAILURE); \
		} \
	} while(0)

int main(int argc, char** argv) {
	check_error(3 == argc, "use: ./umask path mode");
	
	// 保存旧umask并临时设置为0
	mode_t oldUmask = umask(0);
	
	// 使用strtoul转换为无符号长整型，适配mode_t的无符号属性
	char* endptr;
	errno = 0;
	unsigned long mode_val = strtoul(argv[2], &endptr, 8);
	// 检查转换是否成功：无错误且字符串已完全转换
	check_error(errno == 0 && *endptr == '\0', "invalid mode (must be octal, e.g., 0744)");
	mode_t mode = (mode_t)mode_val;
	
	int fd = open(argv[1], O_WRONLY|O_CREAT|O_EXCL, mode);
	if (-1 == fd) {
		if (EEXIST == errno) {
			printf("[file already exists]\n");
			check_error(-1 != chmod(argv[1], mode), "chmod failed");
		} else {
			perror("open failed");
			exit(EXIT_FAILURE);
		}
	} else {
		close(fd);
	}
	
	// 恢复进程原来的umask值
	umask(oldUmask);
	exit(EXIT_SUCCESS);
}

额外建议 #

• 避免在非Unix文件系统上测试Unix权限相关的代码；
• 始终确保权限参数是有效的八进制格式（以0开头），否则转换会出错；
• 对于mode_t这类无符号类型，优先使用strtoul而非strtol进行字符串转换。

内容的提问来源于stack exchange，提问作者alexein777