mirror of
https://github.com/samba-team/samba.git
synced 2024-12-24 21:34:56 +03:00
0d8084ed62
Issue is reported here:
853 case CTDB_CONTROL_DB_VACUUM: {
854 struct ctdb_db_vacuum db_vacuum;
855
>>> CID 1499395: Uninitialized variables (UNINIT)
>>> Using uninitialized value "db_vacuum.full_vacuum_run" when calling "ctdb_db_vacuum_len".
856 CHECK_CONTROL_DATA_SIZE(ctdb_db_vacuum_len(&db_vacuum));
857 return ctdb_control_db_vacuum(ctdb, c, indata, async_reply);
858 }
The problem is that ctdb_bool_len() unnecessarily dereferences its
argument, which in this case is &db_vacuum.full_vacuum_run. Not a
security issue because the value copied by dereferencing is not used.
Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>
Autobuild-User(master): Amitay Isaacs <amitay@samba.org>
Autobuild-Date(master): Wed Feb 23 02:02:06 UTC 2022 on sn-devel-184
|
||
|---|---|---|
| .. | ||
| protocol_api.h | ||
| protocol_basic.c | ||
| protocol_basic.h | ||
| protocol_call.c | ||
| protocol_client.c | ||
| protocol_control.c | ||
| protocol_debug.c | ||
| protocol_header.c | ||
| protocol_keepalive.c | ||
| protocol_message.c | ||
| protocol_packet.c | ||
| protocol_private.h | ||
| protocol_sock.c | ||
| protocol_tunnel.c | ||
| protocol_types.c | ||
| protocol_util.c | ||
| protocol_util.h | ||
| protocol.h | ||