The branch, master has been updated via 6b1f757 s3:lib: use talloc_get_type_abort via 86f04bf lib: Fix CID 1107218 Resource leak via 6bcc037 tevent: Fix CID 1035381 Unchecked return value via 0f91e4e lib: Fix a few CIDs for Resource Leak via b8ac985 ctdb: Fix the O3 developer build from 791c644 Add refentryinfo date.
https://git.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 6b1f757f837f051774132152cb9e80d4c86c161e Author: Ralph Boehme <s...@samba.org> Date: Tue Apr 21 18:58:09 2015 +0200 s3:lib: use talloc_get_type_abort Signed-off-by: Ralph Boehme <s...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> Autobuild-User(master): Michael Adam <ob...@samba.org> Autobuild-Date(master): Tue Apr 21 22:14:07 CEST 2015 on sn-devel-104 commit 86f04bfc248ffcf2d067adcaf65e1569788fb239 Author: Volker Lendecke <v...@samba.org> Date: Tue Mar 31 21:24:45 2015 +0200 lib: Fix CID 1107218 Resource leak Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> commit 6bcc037b5b636e88a914f1d39a95eb6eb72937e0 Author: Volker Lendecke <v...@samba.org> Date: Tue Mar 31 21:15:53 2015 +0200 tevent: Fix CID 1035381 Unchecked return value Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> commit 0f91e4e18db5f896c94f98e84f572568d0b1c3e4 Author: Volker Lendecke <v...@samba.org> Date: Tue Mar 31 09:34:54 2015 +0200 lib: Fix a few CIDs for Resource Leak Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> commit b8ac9853b0483fc4af82f731337464f9b5aaf53c Author: Volker Lendecke <v...@samba.org> Date: Tue Apr 21 10:34:54 2015 +0200 ctdb: Fix the O3 developer build Signed-off-by: Volker Lendecke <v...@samba.org> Reviewed-by: Michael Adam <ob...@samba.org> ----------------------------------------------------------------------- Summary of changes: ctdb/client/ctdb_client.c | 2 +- ctdb/tools/ctdb.c | 20 ++++++++++---------- lib/tevent/testsuite.c | 4 +++- lib/util/tests/file.c | 15 ++++++++++----- source3/lib/popt_common.c | 7 ++----- source3/lib/util_sock.c | 2 +- 6 files changed, 27 insertions(+), 23 deletions(-) Changeset truncated at 500 lines: diff --git a/ctdb/client/ctdb_client.c b/ctdb/client/ctdb_client.c index f46cfc6..6e18269 100644 --- a/ctdb/client/ctdb_client.c +++ b/ctdb/client/ctdb_client.c @@ -3873,7 +3873,7 @@ static bool server_id_exists(struct ctdb_context *ctdb, struct server_id *id) { struct ctdb_server_id sid; int ret; - uint32_t result; + uint32_t result = 0; sid.type = SERVER_TYPE_SAMBA; sid.pnn = id->vnn; diff --git a/ctdb/tools/ctdb.c b/ctdb/tools/ctdb.c index 66d0383..9b7fb11 100644 --- a/ctdb/tools/ctdb.c +++ b/ctdb/tools/ctdb.c @@ -340,7 +340,7 @@ static bool db_exists(struct ctdb_context *ctdb, const char *dbarg, bool dbid_given = false, found = false; uint32_t id; TALLOC_CTX *tmp_ctx = talloc_new(ctdb); - const char *name; + const char *name = NULL; ret = ctdb_ctrl_getdbmap(ctdb, TIMELIMIT(), options.pnn, tmp_ctx, &dbmap); if (ret != 0) { @@ -2803,7 +2803,7 @@ static int unregsrvid(struct ctdb_context *ctdb, int argc, const char **argv) */ static int chksrvid(struct ctdb_context *ctdb, int argc, const char **argv) { - uint32_t status; + uint32_t status = 0; int ret; struct ctdb_server_id server_id; @@ -4589,9 +4589,9 @@ static int control_getdbmap(struct ctdb_context *ctdb, int argc, const char **ar if(options.machinereadable){ printm(":ID:Name:Path:Persistent:Sticky:Unhealthy:ReadOnly:\n"); for(i=0;i<dbmap->num;i++){ - const char *path; - const char *name; - const char *health; + const char *path = NULL; + const char *name = NULL; + const char *health = NULL; bool persistent; bool readonly; bool sticky; @@ -4615,9 +4615,9 @@ static int control_getdbmap(struct ctdb_context *ctdb, int argc, const char **ar printf("Number of databases:%d\n", dbmap->num); for(i=0;i<dbmap->num;i++){ - const char *path; - const char *name; - const char *health; + const char *path = NULL; + const char *name = NULL; + const char *health = NULL; bool persistent; bool readonly; bool sticky; @@ -4647,8 +4647,8 @@ static int control_getdbstatus(struct ctdb_context *ctdb, int argc, const char * const char *db_name; uint32_t db_id; uint8_t flags; - const char *path; - const char *health; + const char *path = NULL; + const char *health = NULL; if (argc < 1) { usage(); diff --git a/lib/tevent/testsuite.c b/lib/tevent/testsuite.c index 34d13bb..e9d1ef4 100644 --- a/lib/tevent/testsuite.c +++ b/lib/tevent/testsuite.c @@ -121,6 +121,7 @@ static bool test_event_context(struct torture_context *test, #endif int finished=0; struct timeval t; + int ret; ev_ctx = tevent_context_init_byname(test, backend); if (ev_ctx == NULL) { @@ -135,7 +136,8 @@ static bool test_event_context(struct torture_context *test, fde_count = 0; /* create a pipe */ - pipe(fd); + ret = pipe(fd); + torture_assert_int_equal(test, ret, 0, "pipe failed"); fde_read = tevent_add_fd(ev_ctx, ev_ctx, fd[0], TEVENT_FD_READ, fde_handler_read, fd); diff --git a/lib/util/tests/file.c b/lib/util/tests/file.c index 8508081..f349c21 100644 --- a/lib/util/tests/file.c +++ b/lib/util/tests/file.c @@ -66,6 +66,7 @@ static bool test_afdgets(struct torture_context *tctx) int fd; char *line; TALLOC_CTX *mem_ctx = tctx; + bool ret = false; torture_assert(tctx, file_save(TEST_FILENAME, (const void *)TEST_DATA, strlen(TEST_DATA)), @@ -76,18 +77,22 @@ static bool test_afdgets(struct torture_context *tctx) torture_assert(tctx, fd != -1, "opening file"); line = afdgets(fd, mem_ctx, 8); - torture_assert(tctx, strcmp(line, TEST_LINE1) == 0, "line 1 mismatch"); + torture_assert_goto(tctx, strcmp(line, TEST_LINE1) == 0, ret, done, + "line 1 mismatch"); line = afdgets(fd, mem_ctx, 8); - torture_assert(tctx, strcmp(line, TEST_LINE2) == 0, "line 2 mismatch"); + torture_assert_goto(tctx, strcmp(line, TEST_LINE2) == 0, ret, done, + "line 2 mismatch"); line = afdgets(fd, mem_ctx, 8); - torture_assert(tctx, strcmp(line, TEST_LINE3) == 0, "line 3 mismatch"); - + torture_assert_goto(tctx, strcmp(line, TEST_LINE3) == 0, ret, done, + "line 3 mismatch"); + ret = true; +done: close(fd); unlink(TEST_FILENAME); - return true; + return ret; } struct torture_suite *torture_local_util_file(TALLOC_CTX *mem_ctx) diff --git a/source3/lib/popt_common.c b/source3/lib/popt_common.c index 6f27bac..44f6aae 100644 --- a/source3/lib/popt_common.c +++ b/source3/lib/popt_common.c @@ -368,11 +368,8 @@ static void popt_common_credentials_callback(poptContext con, } if (getenv("USER")) { - char *puser = SMB_STRDUP(getenv("USER")); - if (!puser) { - exit(ENOMEM); - } - set_cmdline_auth_info_username(auth_info, puser); + set_cmdline_auth_info_username(auth_info, + getenv("USER")); } if (getenv("PASSWD")) { diff --git a/source3/lib/util_sock.c b/source3/lib/util_sock.c index 682d964..c4de85d 100644 --- a/source3/lib/util_sock.c +++ b/source3/lib/util_sock.c @@ -1363,7 +1363,7 @@ struct tevent_req *getaddrinfo_send(TALLOC_CTX *mem_ctx, static void getaddrinfo_do(void *private_data) { struct getaddrinfo_state *state = - (struct getaddrinfo_state *)private_data; + talloc_get_type_abort(private_data, struct getaddrinfo_state); state->ret = getaddrinfo(state->node, state->service, state->hints, &state->res); -- Samba Shared Repository