mirror of
https://github.com/samba-team/samba.git
synced 2025-08-04 08:22:08 +03:00
r7880: fix a typo and memleak on failures cases (patch from marcin)
This commit is contained in:
committed by
Gerald (Jerry) Carter
parent
9af07b2430
commit
6ff0fa0b43
@ -33,7 +33,7 @@ BOOL eventlog_io_q_open_eventlog(const char *desc, EVENTLOG_Q_OPEN_EVENTLOG *q_u
|
||||
if(q_u == NULL)
|
||||
return False;
|
||||
|
||||
/* Data format seems to be:
|
||||
/** Data format seems to be:
|
||||
UNKNOWN structure
|
||||
uint32 unknown
|
||||
uint16 unknown
|
||||
@ -258,7 +258,7 @@ BOOL eventlog_io_q_read_eventlog(const char *desc, EVENTLOG_Q_READ_EVENTLOG *q_u
|
||||
|
||||
return True;
|
||||
}
|
||||
/* Structure of response seems to be:
|
||||
/** Structure of response seems to be:
|
||||
DWORD num_bytes_in_resp -- MUST be the same as q_u->max_read_size
|
||||
for i=0..n
|
||||
EVENTLOGRECORD record
|
||||
@ -397,7 +397,7 @@ BOOL eventlog_io_r_read_eventlog(const char *desc,
|
||||
return True;
|
||||
}
|
||||
|
||||
/* The windows client seems to be doing something funny with the file name
|
||||
/** The windows client seems to be doing something funny with the file name
|
||||
A call like
|
||||
ClearEventLog(handle, "backup_file")
|
||||
on the client side will result in the backup file name looking like this on the
|
||||
|
@ -185,7 +185,7 @@ static BOOL api_eventlog_clear_eventlog(pipes_struct *p)
|
||||
struct api_struct api_eventlog_cmds[] =
|
||||
{
|
||||
{"EVENTLOG_OPENEVENTLOG", EVENTLOG_OPENEVENTLOG, api_eventlog_open_eventlog },
|
||||
{"EVENTLOG_CLOSEVENTLOG", EVENTLOG_CLOSEEVENTLOG, api_eventlog_close_eventlog },
|
||||
{"EVENTLOG_CLOSEEVENTLOG", EVENTLOG_CLOSEEVENTLOG, api_eventlog_close_eventlog },
|
||||
{"EVENTLOG_GETNUMRECORDS", EVENTLOG_GETNUMRECORDS, api_eventlog_get_num_records },
|
||||
{"EVENTLOG_GETOLDESTENTRY", EVENTLOG_GETOLDESTENTRY, api_eventlog_get_oldest_entry },
|
||||
{"EVENTLOG_READEVENTLOG", EVENTLOG_READEVENTLOG, api_eventlog_read_eventlog },
|
||||
|
@ -169,12 +169,18 @@ WERROR _eventlog_open_eventlog(pipes_struct *p,
|
||||
DEBUG(10, ("_eventlog_open_eventlog: Using [%s] as the source log file.\n", info->source_log_file_name));
|
||||
|
||||
if(!create_policy_hnd(p, &(r_u->handle), free_eventlog_info, (void *)info))
|
||||
{
|
||||
free_eventlog_info(info);
|
||||
return WERR_NOMEM;
|
||||
}
|
||||
|
||||
policy_handle_to_string(&r_u->handle, &info->handle_string);
|
||||
|
||||
if(!(_eventlog_open_eventlog_hook(info)))
|
||||
{
|
||||
close_policy_hnd(p, &r_u->handle);
|
||||
return WERR_BADFILE;
|
||||
}
|
||||
|
||||
return WERR_OK;
|
||||
}
|
||||
|
Reference in New Issue
Block a user