On Wed, 08.04.15 20:37, Lennart Poettering (lenn...@poettering.net) wrote: > On Mon, 16.03.15 20:33, Goffredo Baroncelli (kreij...@libero.it) wrote: > > > > > +int change_attr_fd(int fd, unsigned value, unsigned mask) { > > + unsigned old_attr, new_attr; > > + > > + assert(fd >= 0); > > + > > + if (mask == 0) > > + return 0; > > + > > + if (ioctl(fd, FS_IOC_GETFLAGS, &old_attr) < 0) > > + return -errno; > > + > > + new_attr = (old_attr & ~mask) |(value & mask); > > + > > + if (new_attr == old_attr) > > + return 0; > > + > > + if (ioctl(fd, FS_IOC_SETFLAGS, &new_attr) < 0) > > + return -errno; > > + > > + return 0; > > +} > > + > > With this added chattr_fd() is kinda redundant, no?
I fixed this now. Lennart -- Lennart Poettering, Red Hat _______________________________________________ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/systemd-devel