Re: [PATCH] Fix ccsd /etc/cluster/cluster.conf file permissions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



It is creating it with overly-generous permissions, but wouldn't it be
easier to either do something like this when we write the file:

mode_t om;

om = umask(026);
fp = fopen(...);
umask(om);

...or simply call umask(026) when ccsd starts up...?


> -    fp = fopen("/etc/cluster/cluster.conf-update", "w");
> +    cfd = open("/etc/cluster/cluster.conf-update", O_RDWR|O_CREAT, S_IRUSR|S_IWUSR|S_IRGRP);
> +    if(!cfd){
> +      log_sys_err("Unable to open file /etc/cluster/cluster.conf-update");
> +      error = -errno;
> +      goto fail;
> +    }
> +    fp = fdopen(cfd, "w");
>      if(!fp){
> -      log_sys_err("Unable to open /etc/cluster/cluster.conf-update");
> +      log_sys_err("Unable to open stream /etc/cluster/cluster.conf-update");
>        error = -errno;
>        goto fail;
>      }

-- Lon

--

Linux-cluster@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/linux-cluster

[Index of Archives]     [Corosync Cluster Engine]     [GFS]     [Linux Virtualization]     [Centos Virtualization]     [Centos]     [Linux RAID]     [Fedora Users]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite Camping]

  Powered by Linux