免费注册 查看新帖 |

Chinaunix

  平台 论坛 博客 文库
最近访问板块 发新帖
查看: 1829 | 回复: 3
打印 上一主题 下一主题

[C] squid2和squid3原生代码问题 [复制链接]

论坛徽章:
0
跳转到指定楼层
1 [收藏(0)] [报告]
发表于 2013-03-13 16:06 |只看该作者 |倒序浏览
刚在squid2和squid3原生代码中看到下面的用法,怀疑有问题,发出来大家讨论一下:
void
commSetCloseOnExec(int fd)
{
#ifdef FD_CLOEXEC
    int flags;
    int dummy = 0;
    if ((flags = fcntl(fd, F_GETFL, dummy)) < 0)
    {   
        debug(5, 0) ("FD %d: fcntl F_GETFL: %s\n", fd, xstrerror());
        return;
    }   
    if (fcntl(fd, F_SETFD, flags | FD_CLOEXEC) < 0)
        debug(5, 0) ("FD %d: set close-on-exec failed: %s\n", fd, xstrerror());
    fd_table[fd].flags.close_on_exec = 1;
#endif
}

问题如下:
上面的代码是用来设置文件描述符标识的,但是在获取文件描述符标识的时候,使用的是F_GETFL:
flags = fcntl(fd, F_GETFL, dummy))
不因该是F_GETFD吗?

论坛徽章:
0
2 [报告]
发表于 2013-03-13 16:53 |只看该作者
现在文件描述符标识这块只定义了FD_CLOEXEC,不会有什么影响,如果以后有扩展,就会出问题。
但是这种用法是错误的

论坛徽章:
0
3 [报告]
发表于 2013-03-14 10:38 |只看该作者
大家发表一下见解,看看我说的对不对。

论坛徽章:
4
水瓶座
日期:2013-09-06 12:27:30摩羯座
日期:2013-09-28 14:07:46处女座
日期:2013-10-24 14:25:01酉鸡
日期:2014-04-07 11:54:15
4 [报告]
发表于 2013-03-14 11:12 |只看该作者
恩, 应该是楼主说的那样, 看了manpage, 两者应该完全没联系

F_SETFD the file descriptor flags
F_SETFL file status flags

两种概念.
您需要登录后才可以回帖 登录 | 注册

本版积分规则 发表回复

  

北京盛拓优讯信息技术有限公司. 版权所有 京ICP备16024965号-6 北京市公安局海淀分局网监中心备案编号:11010802020122 niuxiaotong@pcpop.com 17352615567
未成年举报专区
中国互联网协会会员  联系我们:huangweiwei@itpub.net
感谢所有关心和支持过ChinaUnix的朋友们 转载本站内容请注明原作者名及出处

清除 Cookies - ChinaUnix - Archiver - WAP - TOP