From patchwork Fri Nov 9 20:04:30 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: John Baldwin X-Patchwork-Id: 30097 Received: (qmail 35243 invoked by alias); 9 Nov 2018 20:04:52 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Delivered-To: mailing list gdb-patches@sourceware.org Received: (qmail 35192 invoked by uid 89); 9 Nov 2018 20:04:48 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-25.1 required=5.0 tests=AWL, BAYES_00, GIT_PATCH_0, GIT_PATCH_1, GIT_PATCH_2, GIT_PATCH_3, SPF_HELO_PASS, SPF_SOFTFAIL autolearn=ham version=3.3.2 spammy=retrieve, sk:gdbarch, 2.18.0, xmalloc X-HELO: mail.baldwin.cx Received: from bigwig.baldwin.cx (HELO mail.baldwin.cx) (96.47.65.170) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 09 Nov 2018 20:04:47 +0000 Received: from ralph.baldwin.cx (ralph.baldwin.cx [66.234.199.215]) by mail.baldwin.cx (Postfix) with ESMTPSA id ED6CB10B707 for ; Fri, 9 Nov 2018 15:04:44 -0500 (EST) From: John Baldwin To: gdb-patches@sourceware.org Subject: [PATCH v3 1/3] Change get_syscalls_by_group to append to an existing vector of integers. Date: Fri, 9 Nov 2018 12:04:30 -0800 Message-Id: <20181109200432.84491-2-jhb@FreeBSD.org> In-Reply-To: <20181109200432.84491-1-jhb@FreeBSD.org> References: <20181109200432.84491-1-jhb@FreeBSD.org> X-IsSubscribed: yes This removes the need for the caller to explicitly manage the memory for the returned system call list. The sole caller only needed the system call numbers rather than the full syscall structures. get_syscalls_by_group now uses a boolean return value to indicate if the requested group exists. gdb/ChangeLog: * break-catch-syscall.c (catch_syscall_split_args): Pass 'result' to get_syscalls_by_group. * xml-syscall.c [!HAVE_LIBEXPAT] (get_syscalls_by_group): Return false. [HAVE_LIBEXPAT] (xml_list_syscalls_by_group): Append syscall numbers to an existing vector of integers and return a bool. (get_syscalls_by_group): Accept reference to vector of integers and change return type to bool. * xml-syscall.h (get_syscalls_by_group): Likewise. --- gdb/ChangeLog | 12 ++++++++++ gdb/break-catch-syscall.c | 14 +----------- gdb/xml-syscall.c | 47 +++++++++++++++------------------------ gdb/xml-syscall.h | 10 ++++----- 4 files changed, 35 insertions(+), 48 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index b6c612bd0b..81faa2994d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,15 @@ +2018-11-09 John Baldwin + + * break-catch-syscall.c (catch_syscall_split_args): Pass 'result' + to get_syscalls_by_group. + * xml-syscall.c [!HAVE_LIBEXPAT] (get_syscalls_by_group): Return + false. + [HAVE_LIBEXPAT] (xml_list_syscalls_by_group): Append syscall + numbers to an existing vector of integers and return a bool. + (get_syscalls_by_group): Accept reference to vector of integers + and change return type to bool. + * xml-syscall.h (get_syscalls_by_group): Likewise. + 2018-11-09 Hafiz Abid Qadeer * configure: Regenerate. diff --git a/gdb/break-catch-syscall.c b/gdb/break-catch-syscall.c index 93ef74c249..f3740a1673 100644 --- a/gdb/break-catch-syscall.c +++ b/gdb/break-catch-syscall.c @@ -409,25 +409,13 @@ catch_syscall_split_args (const char *arg) { /* We have a syscall group. Let's expand it into a syscall list before inserting. */ - struct syscall *syscall_list; const char *group_name; /* Skip over "g:" and "group:" prefix strings. */ group_name = strchr (cur_name, ':') + 1; - syscall_list = get_syscalls_by_group (gdbarch, group_name); - - if (syscall_list == NULL) + if (!get_syscalls_by_group (gdbarch, group_name, result)) error (_("Unknown syscall group '%s'."), group_name); - - for (i = 0; syscall_list[i].name != NULL; i++) - { - /* Insert each syscall that are part of the group. No - need to check if it is valid. */ - result.push_back (syscall_list[i].number); - } - - xfree (syscall_list); } else { diff --git a/gdb/xml-syscall.c b/gdb/xml-syscall.c index bf17642911..664cee77dd 100644 --- a/gdb/xml-syscall.c +++ b/gdb/xml-syscall.c @@ -77,11 +77,12 @@ get_syscall_names (struct gdbarch *gdbarch) return NULL; } -struct syscall * -get_syscalls_by_group (struct gdbarch *gdbarch, const char *group) +bool +get_syscalls_by_group (struct gdbarch *gdbarch, const char *group, + std::vector &syscall_numbers) { syscall_warn_user (); - return NULL; + return false; } const char ** @@ -444,40 +445,27 @@ xml_list_of_syscalls (struct gdbarch *gdbarch) } /* Iterate over the syscall_group_desc element to return a list of - syscalls that are part of the given group, terminated by an empty - element. If the syscall group doesn't exist, return NULL. */ + syscalls that are part of the given group. If the syscall group + doesn't exist, return false. */ -static struct syscall * -xml_list_syscalls_by_group (struct gdbarch *gdbarch, const char *group) +static bool +xml_list_syscalls_by_group (struct gdbarch *gdbarch, const char *group, + std::vector &syscalls) { struct syscalls_info *syscalls_info = gdbarch_syscalls_info (gdbarch); struct syscall_group_desc *groupdesc; - struct syscall *syscalls = NULL; - int nsyscalls; - int i; if (syscalls_info == NULL) - return NULL; + return false; groupdesc = syscall_group_get_group_by_name (syscalls_info, group); if (groupdesc == NULL) - return NULL; + return false; - nsyscalls = groupdesc->syscalls.size (); - syscalls = (struct syscall*) xmalloc ((nsyscalls + 1) - * sizeof (struct syscall)); + for (const struct syscall_desc *sysdesc : groupdesc->syscalls) + syscalls.push_back (sysdesc->number); - for (i = 0; i < groupdesc->syscalls.size (); i++) - { - syscalls[i].name = groupdesc->syscalls[i]->name.c_str (); - syscalls[i].number = groupdesc->syscalls[i]->number; - } - - /* Add final element marker. */ - syscalls[i].name = NULL; - syscalls[i].number = 0; - - return syscalls; + return true; } /* Return a NULL terminated list of syscall groups or an empty list, if @@ -542,12 +530,13 @@ get_syscall_names (struct gdbarch *gdbarch) /* See comment in xml-syscall.h. */ -struct syscall * -get_syscalls_by_group (struct gdbarch *gdbarch, const char *group) +bool +get_syscalls_by_group (struct gdbarch *gdbarch, const char *group, + std::vector &syscall_numbers) { init_syscalls_info (gdbarch); - return xml_list_syscalls_by_group (gdbarch, group); + return xml_list_syscalls_by_group (gdbarch, group, syscall_numbers); } /* See comment in xml-syscall.h. */ diff --git a/gdb/xml-syscall.h b/gdb/xml-syscall.h index 4429d66400..2e76a9a97f 100644 --- a/gdb/xml-syscall.h +++ b/gdb/xml-syscall.h @@ -51,13 +51,11 @@ void get_syscall_by_name (struct gdbarch *gdbarch, const char **get_syscall_names (struct gdbarch *gdbarch); /* Function used to retrieve the list of syscalls of a given group in - the system. Return a list of syscalls that are element of the - group, terminated by an empty element. The list is malloc'ed - and must be freed by the caller. If group doesn't exist, return - NULL. */ + the system. The syscall numbers are appended to SYSCALL_NUMBERS. + If the group doesn't exist, return false. */ -struct syscall *get_syscalls_by_group (struct gdbarch *gdbarch, - const char *group); +bool get_syscalls_by_group (struct gdbarch *gdbarch, const char *group, + std::vector &syscall_numbers); /* Function used to retrieve the list of syscall groups in the system. Return an array of strings terminated by a NULL element. The list