qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 0/1] Fix bochs memory leak
@ 2019-12-10 21:13 Cameron Esfahani via
  2019-12-10 21:13 ` [PATCH v1 1/1] display/bochs-display: fix " Cameron Esfahani via
  2019-12-10 21:23 ` [PATCH v1 0/1] Fix bochs " no-reply
  0 siblings, 2 replies; 3+ messages in thread
From: Cameron Esfahani via @ 2019-12-10 21:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: kraxel

Fix a small memory leak in the Bochs display driver.

Each frame would leak about 304 bytes.

Cameron Esfahani (1):
  display/bochs-display: fix memory leak

 hw/display/bochs-display.c | 2 ++
 1 file changed, 2 insertions(+)

-- 
2.24.0



^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH v1 1/1] display/bochs-display: fix memory leak
  2019-12-10 21:13 [PATCH v1 0/1] Fix bochs memory leak Cameron Esfahani via
@ 2019-12-10 21:13 ` Cameron Esfahani via
  2019-12-10 21:23 ` [PATCH v1 0/1] Fix bochs " no-reply
  1 sibling, 0 replies; 3+ messages in thread
From: Cameron Esfahani via @ 2019-12-10 21:13 UTC (permalink / raw)
  To: qemu-devel; +Cc: kraxel

Fix memory leak in bochs_display_update().  Leaks 304 bytes per frame.
---
 hw/display/bochs-display.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/hw/display/bochs-display.c b/hw/display/bochs-display.c
index dc1bd1641d..215db9a231 100644
--- a/hw/display/bochs-display.c
+++ b/hw/display/bochs-display.c
@@ -252,6 +252,8 @@ static void bochs_display_update(void *opaque)
             dpy_gfx_update(s->con, 0, ys,
                            mode.width, y - ys);
         }
+
+        g_free(snap);
     }
 }
 
-- 
2.24.0



^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v1 0/1] Fix bochs memory leak
  2019-12-10 21:13 [PATCH v1 0/1] Fix bochs memory leak Cameron Esfahani via
  2019-12-10 21:13 ` [PATCH v1 1/1] display/bochs-display: fix " Cameron Esfahani via
@ 2019-12-10 21:23 ` no-reply
  1 sibling, 0 replies; 3+ messages in thread
From: no-reply @ 2019-12-10 21:23 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-devel, kraxel

Patchew URL: https://patchew.org/QEMU/cover.1576012190.git.dirty@apple.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Subject: [PATCH v1 0/1] Fix bochs memory leak
Type: series
Message-id: cover.1576012190.git.dirty@apple.com

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Switched to a new branch 'test'
a3c9e5d display/bochs-display: fix memory leak

=== OUTPUT BEGIN ===
ERROR: Missing Signed-off-by: line(s)

total: 1 errors, 0 warnings, 8 lines checked

Commit a3c9e5dc8c32 (display/bochs-display: fix memory leak) has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/cover.1576012190.git.dirty@apple.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-12-10 21:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-10 21:13 [PATCH v1 0/1] Fix bochs memory leak Cameron Esfahani via
2019-12-10 21:13 ` [PATCH v1 1/1] display/bochs-display: fix " Cameron Esfahani via
2019-12-10 21:23 ` [PATCH v1 0/1] Fix bochs " no-reply

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).