mirror of
https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git
synced 2025-12-31 21:50:11 +01:00
dhcp: remove local modifications initializing cleanup variables
I cannot reproduce any compiler warnings with these changes. I think it is desirable, that our version of the code is as similar as possible to the upstream systemd code. Undo these local modifications.
This commit is contained in:
parent
746d0f544c
commit
4a58425dbf
5 changed files with 8 additions and 6 deletions
|
|
@ -401,7 +401,7 @@ static void client_stop(sd_dhcp_client *client, int error) {
|
|||
|
||||
static int client_message_init(sd_dhcp_client *client, DHCPPacket **ret,
|
||||
uint8_t type, size_t *_optlen, size_t *_optoffset) {
|
||||
_cleanup_free_ DHCPPacket *packet = NULL;
|
||||
_cleanup_free_ DHCPPacket *packet;
|
||||
size_t optlen, optoffset, size;
|
||||
be16_t max_size;
|
||||
usec_t time_now;
|
||||
|
|
|
|||
|
|
@ -672,7 +672,7 @@ int sd_dhcp_lease_save(sd_dhcp_lease *lease, const char *lease_file) {
|
|||
|
||||
r = sd_dhcp_lease_get_client_id(lease, &client_id, &client_id_len);
|
||||
if (r >= 0) {
|
||||
_cleanup_free_ char *client_id_hex = NULL;
|
||||
_cleanup_free_ char *client_id_hex;
|
||||
|
||||
client_id_hex = hexmem (client_id, client_id_len);
|
||||
if (!client_id_hex) {
|
||||
|
|
|
|||
|
|
@ -846,7 +846,7 @@ static int client_receive_message(sd_event_source *s, int fd, uint32_t revents,
|
|||
void *userdata) {
|
||||
sd_dhcp6_client *client = userdata;
|
||||
DHCP6_CLIENT_DONT_DESTROY(client);
|
||||
_cleanup_free_ DHCP6Message *message = NULL;
|
||||
_cleanup_free_ DHCP6Message *message;
|
||||
int r, buflen, len;
|
||||
|
||||
assert(s);
|
||||
|
|
|
|||
|
|
@ -510,15 +510,17 @@ static int parse_env_file_push(
|
|||
va_list aq, *ap = userdata;
|
||||
|
||||
if (!utf8_is_valid(key)) {
|
||||
_cleanup_free_ char *p = utf8_escape_invalid(key);
|
||||
_cleanup_free_ char *p;
|
||||
|
||||
p = utf8_escape_invalid(key);
|
||||
log_error("%s:%u: invalid UTF-8 in key '%s', ignoring.", strna(filename), line, p);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
||||
if (value && !utf8_is_valid(value)) {
|
||||
_cleanup_free_ char *p = utf8_escape_invalid(value);
|
||||
_cleanup_free_ char *p;
|
||||
|
||||
p = utf8_escape_invalid(value);
|
||||
log_error("%s:%u: invalid UTF-8 value for key %s: '%s', ignoring.", strna(filename), line, key, p);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -3694,7 +3694,7 @@ char *ellipsize(const char *s, size_t length, unsigned percent) {
|
|||
}
|
||||
|
||||
int touch_file(const char *path, bool parents, usec_t stamp, uid_t uid, gid_t gid, mode_t mode) {
|
||||
_cleanup_close_ int fd = -1;
|
||||
_cleanup_close_ int fd;
|
||||
int r;
|
||||
|
||||
assert(path);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue