[PATCH 0/8] Replacing malloc and the like with GLib's variants

Mahmoud Mandour posted 8 patches 3 years, 1 month ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20210314032324.45142-1-ma.mandourr@gmail.com
Maintainers: Warner Losh <imp@bsdimp.com>, Stefan Hajnoczi <stefanha@redhat.com>, Max Filippov <jcmvbkbc@gmail.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Kyle Evans <kevans@freebsd.org>, Gerd Hoffmann <kraxel@redhat.com>
bsd-user/elfload.c               | 74 ++++++++++++++++----------------
hw/audio/fmopl.c                 | 68 +++++++++++++++++------------
target/xtensa/xtensa-isa.c       | 53 +++++++++--------------
tools/virtiofsd/buffer.c         |  4 +-
tools/virtiofsd/fuse_lowlevel.c  | 30 ++++++-------
tools/virtiofsd/fuse_opt.c       |  4 +-
tools/virtiofsd/fuse_virtio.c    | 34 +++++++--------
tools/virtiofsd/passthrough_ll.c | 32 +++++++-------
util/compatfd.c                  | 10 ++---
9 files changed, 156 insertions(+), 153 deletions(-)
[PATCH 0/8] Replacing malloc and the like with GLib's variants
Posted by Mahmoud Mandour 3 years, 1 month ago
Inspired by the task in the Bite-sized tasks page, I've tried to change
some of the occurrences of malloc(), calloc(), and realloc() calls and
their respective free() calls. I also included some minor style
improvements that caused errors and warnings when supplied to the 
checkpatch.pl script.

At numerous places, the change is quite obvious. However, some other 
malloc() and the like have no immediately visible free() calls. Analysis
was done to trace the roots of each free() call to match each GLib function
with a g_free(), and every malloc(), realloc(), or calloc() call with a
call to free().

This series was resent due to some previous problems that are hopefully
resolved in this series.

Mahmoud Mandour (8):
  bsd-user/elfload.c: Replaced calls to malloc/free with GLib variants
  hw/audio/fmopl.c: Fixing some style errors
  hw/audio/fmopl.c: Replaced calls to malloc with GLib's variants
  target/xtensa: Replaced malloc/free with GLib's variants
  util/compatfd.c: Replaced a malloc with GLib's variant
  tools/virtiofsd/buffer.c: replaced a calloc call with GLib's
    g_try_new0
  tools/virtiofsd/fuse_opt.c: Replaced a malloc with GLib's g_try_malloc
  tools/virtiofsd: Replacing malloc-like calls with GLib's variants

 bsd-user/elfload.c               | 74 ++++++++++++++++----------------
 hw/audio/fmopl.c                 | 68 +++++++++++++++++------------
 target/xtensa/xtensa-isa.c       | 53 +++++++++--------------
 tools/virtiofsd/buffer.c         |  4 +-
 tools/virtiofsd/fuse_lowlevel.c  | 30 ++++++-------
 tools/virtiofsd/fuse_opt.c       |  4 +-
 tools/virtiofsd/fuse_virtio.c    | 34 +++++++--------
 tools/virtiofsd/passthrough_ll.c | 32 +++++++-------
 util/compatfd.c                  | 10 ++---
 9 files changed, 156 insertions(+), 153 deletions(-)

-- 
2.25.1