mirror of
https://github.com/systemd/systemd.git
synced 2024-10-28 20:25:38 +03:00
udev,update-done: more log_xyz_errno() conversions
This commit is contained in:
parent
1797280193
commit
755bde375f
@ -342,14 +342,14 @@ int link_config_apply(link_config_ctx *ctx, link_config *config,
|
|||||||
r = ethtool_set_speed(&ctx->ethtool_fd, old_name, config->speed / 1024,
|
r = ethtool_set_speed(&ctx->ethtool_fd, old_name, config->speed / 1024,
|
||||||
config->duplex);
|
config->duplex);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
log_warning("Could not set speed or duplex of %s to %u Mbps (%s): %s",
|
log_warning_errno(r, "Could not set speed or duplex of %s to %u Mbps (%s): %m",
|
||||||
old_name, config->speed / 1024,
|
old_name, config->speed / 1024,
|
||||||
duplex_to_string(config->duplex), strerror(-r));
|
duplex_to_string(config->duplex));
|
||||||
|
|
||||||
r = ethtool_set_wol(&ctx->ethtool_fd, old_name, config->wol);
|
r = ethtool_set_wol(&ctx->ethtool_fd, old_name, config->wol);
|
||||||
if (r < 0)
|
if (r < 0)
|
||||||
log_warning("Could not set WakeOnLan of %s to %s: %s",
|
log_warning_errno(r, "Could not set WakeOnLan of %s to %s: %m",
|
||||||
old_name, wol_to_string(config->wol), strerror(-r));
|
old_name, wol_to_string(config->wol));
|
||||||
|
|
||||||
ifindex = udev_device_get_ifindex(device);
|
ifindex = udev_device_get_ifindex(device);
|
||||||
if (ifindex <= 0) {
|
if (ifindex <= 0) {
|
||||||
@ -422,8 +422,7 @@ int link_config_apply(link_config_ctx *ctx, link_config *config,
|
|||||||
r = rtnl_set_link_properties(&ctx->rtnl, ifindex, config->alias, mac,
|
r = rtnl_set_link_properties(&ctx->rtnl, ifindex, config->alias, mac,
|
||||||
config->mtu);
|
config->mtu);
|
||||||
if (r < 0) {
|
if (r < 0) {
|
||||||
log_warning("Could not set Alias, MACAddress or MTU on %s: %s",
|
log_warning_errno(r, "Could not set Alias, MACAddress or MTU on %s: %m", old_name);
|
||||||
old_name, strerror(-r));
|
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -776,8 +776,7 @@ static int rename_netif(struct udev_event *event) {
|
|||||||
|
|
||||||
r = rtnl_set_link_name(&event->rtnl, udev_device_get_ifindex(dev), name);
|
r = rtnl_set_link_name(&event->rtnl, udev_device_get_ifindex(dev), name);
|
||||||
if (r < 0) {
|
if (r < 0) {
|
||||||
log_error("error changing net interface name '%s' to '%s': %s",
|
log_error_errno(r, "Error changing net interface name '%s' to '%s': %m", oldname, name);
|
||||||
oldname, name, strerror(-r));
|
|
||||||
return r;
|
return r;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -46,13 +46,10 @@ static int apply_timestamp(const char *path, struct timespec *ts) {
|
|||||||
|
|
||||||
if (utimensat(AT_FDCWD, path, twice, AT_SYMLINK_NOFOLLOW) < 0) {
|
if (utimensat(AT_FDCWD, path, twice, AT_SYMLINK_NOFOLLOW) < 0) {
|
||||||
|
|
||||||
if (errno == EROFS) {
|
if (errno == EROFS)
|
||||||
log_debug("Can't update timestamp file %s, file system is read-only.", path);
|
return log_debug("Can't update timestamp file %s, file system is read-only.", path);
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
log_error("Failed to update timestamp on %s: %m", path);
|
return log_error_errno(errno, "Failed to update timestamp on %s: %m", path);
|
||||||
return -errno;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
} else if (errno == ENOENT) {
|
} else if (errno == ENOENT) {
|
||||||
@ -62,24 +59,17 @@ static int apply_timestamp(const char *path, struct timespec *ts) {
|
|||||||
/* The timestamp file doesn't exist yet? Then let's create it. */
|
/* The timestamp file doesn't exist yet? Then let's create it. */
|
||||||
|
|
||||||
r = mac_selinux_create_file_prepare(path, S_IFREG);
|
r = mac_selinux_create_file_prepare(path, S_IFREG);
|
||||||
if (r < 0) {
|
if (r < 0)
|
||||||
log_error("Failed to set SELinux context for %s: %s",
|
return log_error_errno(r, "Failed to set SELinux context for %s: %m", path);
|
||||||
path, strerror(-r));
|
|
||||||
return r;
|
|
||||||
}
|
|
||||||
|
|
||||||
fd = open(path, O_CREAT|O_EXCL|O_WRONLY|O_TRUNC|O_CLOEXEC|O_NOCTTY|O_NOFOLLOW, 0644);
|
fd = open(path, O_CREAT|O_EXCL|O_WRONLY|O_TRUNC|O_CLOEXEC|O_NOCTTY|O_NOFOLLOW, 0644);
|
||||||
mac_selinux_create_file_clear();
|
mac_selinux_create_file_clear();
|
||||||
|
|
||||||
if (fd < 0) {
|
if (fd < 0) {
|
||||||
|
if (errno == EROFS)
|
||||||
|
return log_debug("Can't create timestamp file %s, file system is read-only.", path);
|
||||||
|
|
||||||
if (errno == EROFS) {
|
return log_error_errno(errno, "Failed to create timestamp file %s: %m", path);
|
||||||
log_debug("Can't create timestamp file %s, file system is read-only.", path);
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
log_error("Failed to create timestamp file %s: %m", path);
|
|
||||||
return -errno;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
(void) loop_write(fd, MESSAGE, strlen(MESSAGE), false);
|
(void) loop_write(fd, MESSAGE, strlen(MESSAGE), false);
|
||||||
@ -87,14 +77,10 @@ static int apply_timestamp(const char *path, struct timespec *ts) {
|
|||||||
twice[0] = *ts;
|
twice[0] = *ts;
|
||||||
twice[1] = *ts;
|
twice[1] = *ts;
|
||||||
|
|
||||||
if (futimens(fd, twice) < 0) {
|
if (futimens(fd, twice) < 0)
|
||||||
log_error("Failed to update timestamp on %s: %m", path);
|
return log_error_errno(errno, "Failed to update timestamp on %s: %m", path);
|
||||||
return -errno;
|
} else
|
||||||
}
|
log_error_errno(errno, "Failed to stat() timestamp file %s: %m", path);
|
||||||
} else {
|
|
||||||
log_error("Failed to stat() timestamp file %s: %m", path);
|
|
||||||
return -errno;
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
@ -108,7 +94,7 @@ int main(int argc, char *argv[]) {
|
|||||||
log_open();
|
log_open();
|
||||||
|
|
||||||
if (stat("/usr", &st) < 0) {
|
if (stat("/usr", &st) < 0) {
|
||||||
log_error("Failed to stat /usr: %m");
|
log_error_errno(errno, "Failed to stat /usr: %m");
|
||||||
return EXIT_FAILURE;
|
return EXIT_FAILURE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user