Re: [PATCH spice-server] tests: Use GLib memory functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>



On Thu, 2017-11-30 at 11:15 +0000, Frediano Ziglio wrote:
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/tests/test-display-base.c         | 42 ++++++++++++++++----
> ------------
>  server/tests/test-display-streaming.c    |  2 +-
>  server/tests/test-display-width-stride.c |  4 +--
>  server/tests/test-gst.c                  | 12 ++++-----
>  server/tests/test-qxl-parsing.c          | 18 +++++++-------
>  server/tests/test-stat-file.c            |  4 +--
>  6 files changed, 41 insertions(+), 41 deletions(-)
> 
> diff --git a/server/tests/test-display-base.c b/server/tests/test-
> display-base.c
> index 83eb92e5..2677eb42 100644
> --- a/server/tests/test-display-base.c
> +++ b/server/tests/test-display-base.c
> @@ -65,10 +65,10 @@ static void
> test_spice_destroy_update(SimpleSpiceUpdate *update)
>      }
>      if (update->drawable.clip.type != SPICE_CLIP_TYPE_NONE) {
>          uint8_t *ptr = (uint8_t*)update->drawable.clip.data;
> -        free(ptr);
> +        g_free(ptr);
>      }
> -    free(update->bitmap);
> -    free(update);
> +    g_free(update->bitmap);
> +    g_free(update);
>  }
>  
>  #define DEFAULT_WIDTH 640
> @@ -186,7 +186,7 @@ test_spice_create_update_from_bitmap(uint32_t
> surface_id,
>      bh = bbox.bottom - bbox.top;
>      bw = bbox.right - bbox.left;
>  
> -    update   = spice_new0(SimpleSpiceUpdate, 1);
> +    update   = g_new0(SimpleSpiceUpdate, 1);
>      update->bitmap = bitmap;
>      drawable = &update->drawable;
>      image    = &update->image;
> @@ -199,7 +199,7 @@ test_spice_create_update_from_bitmap(uint32_t
> surface_id,
>      } else {
>          QXLClipRects *cmd_clip;
>  
> -        cmd_clip = spice_malloc0(sizeof(QXLClipRects) +
> num_clip_rects*sizeof(QXLRect));
> +        cmd_clip = g_malloc0(sizeof(QXLClipRects) +
> num_clip_rects*sizeof(QXLRect));
>          cmd_clip->num_rects = num_clip_rects;
>          cmd_clip->chunk.data_size = num_clip_rects*sizeof(QXLRect);
>          cmd_clip->chunk.prev_chunk = cmd_clip->chunk.next_chunk = 0;
> @@ -208,7 +208,7 @@ test_spice_create_update_from_bitmap(uint32_t
> surface_id,
>          drawable->clip.type = SPICE_CLIP_TYPE_RECTS;
>          drawable->clip.data = (intptr_t)cmd_clip;
>  
> -        free(clip_rects);
> +        g_free(clip_rects);
>      }
>      drawable->effect          = QXL_EFFECT_OPAQUE;
>      simple_set_release_info(&drawable->release_info,
> (intptr_t)update);
> @@ -248,7 +248,7 @@ static SimpleSpiceUpdate
> *test_spice_create_update_solid(uint32_t surface_id, QX
>      bw = bbox.right - bbox.left;
>      bh = bbox.bottom - bbox.top;
>  
> -    bitmap = spice_malloc(bw * bh * 4);
> +    bitmap = g_malloc(bw * bh * 4);
>      dst = (uint32_t *)bitmap;
>  
>      for (i = 0 ; i < bh * bw ; ++i, ++dst) {
> @@ -283,7 +283,7 @@ static SimpleSpiceUpdate
> *test_spice_create_update_draw(Test *test, uint32_t sur
>      bw       = test->primary_width/SINGLE_PART;
>      bh       = 48;
>  
> -    bitmap = dst = spice_malloc(bw * bh * 4);
> +    bitmap = dst = g_malloc(bw * bh * 4);
>      //printf("allocated %p\n", dst);
>  
>      for (i = 0 ; i < bh * bw ; ++i, dst+=4) {
> @@ -308,7 +308,7 @@ static SimpleSpiceUpdate
> *test_spice_create_update_copy_bits(Test *test, uint32_
>          .top = 0,
>      };
>  
> -    update   = spice_new0(SimpleSpiceUpdate, 1);
> +    update   = g_new0(SimpleSpiceUpdate, 1);
>      drawable = &update->drawable;
>  
>      bw       = test->primary_width/SINGLE_PART;
> @@ -353,7 +353,7 @@ static int format_to_bpp(int format)
>  
>  static SimpleSurfaceCmd *create_surface(int surface_id, int format,
> int width, int height, uint8_t *data)
>  {
> -    SimpleSurfaceCmd *simple_cmd = spice_new0(SimpleSurfaceCmd, 1);
> +    SimpleSurfaceCmd *simple_cmd = g_new0(SimpleSurfaceCmd, 1);
>      QXLSurfaceCmd *surface_cmd = &simple_cmd->surface_cmd;
>      int bpp = format_to_bpp(format);
>  
> @@ -372,7 +372,7 @@ static SimpleSurfaceCmd *create_surface(int
> surface_id, int format, int width, i
>  
>  static SimpleSurfaceCmd *destroy_surface(int surface_id)
>  {
> -    SimpleSurfaceCmd *simple_cmd = spice_new0(SimpleSurfaceCmd, 1);
> +    SimpleSurfaceCmd *simple_cmd = g_new0(SimpleSurfaceCmd, 1);
>      QXLSurfaceCmd *surface_cmd = &simple_cmd->surface_cmd;
>  
>      set_cmd(&simple_cmd->ext, QXL_CMD_SURFACE,
> (intptr_t)surface_cmd);
> @@ -675,14 +675,14 @@ static void release_resource(SPICE_GNUC_UNUSED
> QXLInstance *qin,
>              test_spice_destroy_update((void*)ext);
>              break;
>          case QXL_CMD_SURFACE:
> -            free(ext);
> +            g_free(ext);
>              break;
>          case QXL_CMD_CURSOR: {
>              QXLCursorCmd *cmd = (QXLCursorCmd *)(uintptr_t)ext-
> >cmd.data;
>              if (cmd->type == QXL_CURSOR_SET || cmd->type ==
> QXL_CURSOR_MOVE) {
> -                free(cmd);
> +                g_free(cmd);
>              }
> -            free(ext);
> +            g_free(ext);
>              break;
>          }
>          default:
> @@ -731,8 +731,8 @@ static int get_cursor_command(QXLInstance *qin,
> struct QXLCommandExt *ext)
>      }
>  
>      test->cursor_notify--;
> -    cmd = spice_new0(QXLCommandExt, 1);
> -    cursor_cmd = spice_new0(QXLCursorCmd, 1);
> +    cmd = g_new0(QXLCommandExt, 1);
> +    cursor_cmd = g_new0(QXLCursorCmd, 1);
>  
>      cursor_cmd->release_info.id = (uintptr_t)cmd;
>  
> @@ -886,8 +886,8 @@ void test_set_simple_command_list(Test *test,
> const int *simple_commands, int nu
>  {
>      int i;
>  
> -    free(test->commands);
> -    test->commands = spice_new0(Command, num_commands);
> +    g_free(test->commands);
> +    test->commands = g_new0(Command, num_commands);
>      test->num_commands = num_commands;
>      for (i = 0 ; i < num_commands; ++i) {
>          test->commands[i].command = simple_commands[i];
> @@ -923,7 +923,7 @@ static gboolean ignore_bind_failures(const gchar
> *log_domain,
>  
>  Test* test_new(SpiceCoreInterface* core)
>  {
> -    Test *test = spice_new0(Test, 1);
> +    Test *test = g_new0(Test, 1);
>      int port = -1;
>  
>      test->qxl_instance.base.sif = &display_sif.base;
> @@ -965,8 +965,8 @@ void test_destroy(Test *test)
>      // this timer is used by spice server so
>      // avoid to free it while is running
>      test->core->timer_remove(test->wakeup_timer);
> -    free(test->commands);
> -    free(test);
> +    g_free(test->commands);
> +    g_free(test);
>  }
>  
>  static void init_automated(void)
> diff --git a/server/tests/test-display-streaming.c
> b/server/tests/test-display-streaming.c
> index a881f11e..1cd3f6e5 100644
> --- a/server/tests/test-display-streaming.c
> +++ b/server/tests/test-display-streaming.c
> @@ -151,7 +151,7 @@ static void create_clipped_frame(Test *test,
> Command *command, int clipping_fact
>          cmd->num_clip_rects = 0;
>      } else {
>          cmd->num_clip_rects = 2;
> -        cmd->clip_rects = calloc(sizeof(QXLRect), 2);
> +        cmd->clip_rects = g_new0(QXLRect, 2);
>          cmd->clip_rects[0].left = OVERLAY_WIDTH;
>          cmd->clip_rects[0].top = cmd->bbox.top;
>          cmd->clip_rects[0].right = cmd->bbox.right -
> clipping_factor;
> diff --git a/server/tests/test-display-width-stride.c
> b/server/tests/test-display-width-stride.c
> index 711cb664..5576f7dd 100644
> --- a/server/tests/test-display-width-stride.c
> +++ b/server/tests/test-display-width-stride.c
> @@ -64,7 +64,7 @@ set_surface_params(SPICE_GNUC_UNUSED Test *test,
> Command *command)
>      create->format = SPICE_SURFACE_FMT_8_A;
>      create->width = 128;
>      create->height = 128;
> -    g_surface_data = realloc(g_surface_data, create->width * create-
> >height * 1);
> +    g_surface_data = g_realloc(g_surface_data, create->width *
> create->height * 1);
>      create->surface_id = g_surface_id;
>      create->data = g_surface_data;
>  }
> @@ -73,7 +73,7 @@ static void
>  set_destroy_parameters(SPICE_GNUC_UNUSED Test *test,
> SPICE_GNUC_UNUSED Command *command)
>  {
>      if (g_surface_data) {
> -        free(g_surface_data);
> +        g_free(g_surface_data);
>          g_surface_data = NULL;
>      }
>  }
> diff --git a/server/tests/test-gst.c b/server/tests/test-gst.c
> index 69633e39..ecd6c49a 100644
> --- a/server/tests/test-gst.c
> +++ b/server/tests/test-gst.c
> @@ -585,7 +585,7 @@ handle_pipeline_message(GstBus *bus, GstMessage
> *msg, gpointer test_pipeline)
>  static TestPipeline*
>  create_pipeline(const char *desc, SampleProc sample_proc, void
> *param)
>  {
> -    TestPipeline *pipeline = spice_new0(TestPipeline, 1);
> +    TestPipeline *pipeline = g_new0(TestPipeline, 1);
>  
>      pipeline->sample_proc = sample_proc;
>      pipeline->sample_param = param;
> @@ -630,7 +630,7 @@ pipeline_free(TestPipeline *pipeline)
>      }
>      gst_object_unref(pipeline->appsink);
>      gst_object_unref(pipeline->gst_pipeline);
> -    free(pipeline);
> +    g_free(pipeline);
>  }
>  
>  static void
> @@ -755,7 +755,7 @@ frame_unref(TestFrame *frame)
>          return;
>      }
>      bitmap_free(frame->bitmap);
> -    free(frame);
> +    g_free(frame);
>  }
>  
>  static void
> @@ -767,7 +767,7 @@ bitmap_free(SpiceBitmap *bitmap)
>      spice_assert(!bitmap->palette);
>      spice_assert(bitmap->data);
>      spice_chunks_destroy(bitmap->data);
> -    free(bitmap);
> +    g_free(bitmap);
>  }
>  
>  static SpiceChunks* chunks_alloc(uint32_t stride, uint32_t height,
> uint32_t split);
> @@ -792,7 +792,7 @@ gst_to_spice_bitmap(GstSample *sample)
>      spice_assert(gst_structure_get_int(s, "width", &width) &&
>                   gst_structure_get_int(s, "height", &height));
>  
> -    SpiceBitmap *bitmap = spice_new0(SpiceBitmap, 1);
> +    SpiceBitmap *bitmap = g_new0(SpiceBitmap, 1);
>      bitmap->format = bitmap_format;
>      bitmap->flags = top_down ? SPICE_BITMAP_FLAGS_TOP_DOWN : 0;
>      bitmap->x = width;
> @@ -953,7 +953,7 @@ get_bitmap_format(const char *format)
>  static TestFrame *
>  gst_to_spice_frame(GstSample *sample)
>  {
> -    TestFrame *frame = spice_new0(TestFrame, 1);
> +    TestFrame *frame = g_new0(TestFrame, 1);
>      frame->refs = 1;
>      frame->bitmap = gst_to_spice_bitmap(sample);
>      return frame;
> diff --git a/server/tests/test-qxl-parsing.c b/server/tests/test-qxl-
> parsing.c
> index 9c0c3b1c..47139a48 100644
> --- a/server/tests/test-qxl-parsing.c
> +++ b/server/tests/test-qxl-parsing.c
> @@ -48,7 +48,7 @@ from_physical(QXLPHYSICAL physical)
>  static void*
>  create_chunk(size_t prefix, uint32_t size, QXLDataChunk* prev, int
> fill)
>  {
> -    uint8_t *ptr = spice_malloc0(prefix + sizeof(QXLDataChunk) +
> size);
> +    uint8_t *ptr = g_malloc0(prefix + sizeof(QXLDataChunk) + size);
>      QXLDataChunk *qxl = (QXLDataChunk *) (ptr + prefix);
>      memset(&qxl->data[0], fill, size);
>      qxl->data_size = size;
> @@ -76,13 +76,13 @@ static void init_qxl_surface(QXLSurfaceCmd *qxl)
>      qxl->u.surface_create.width = 128;
>      qxl->u.surface_create.stride = 512;
>      qxl->u.surface_create.height = 128;
> -    surface_mem = malloc(0x10000);
> +    surface_mem = g_malloc(0x10000);
>      qxl->u.surface_create.data = to_physical(surface_mem);
>  }
>  
>  static void deinit_qxl_surface(QXLSurfaceCmd *qxl)
>  {
> -    free(from_physical(qxl->u.surface_create.data));
> +    g_free(from_physical(qxl->u.surface_create.data));
>  }
>  
>  static void test_no_issues(void)
> @@ -168,8 +168,8 @@ static void test_cursor_command(void)
>      cursor_cmd.u.set.shape = to_physical(cursor);
>  
>      g_assert_true(red_get_cursor_cmd(&mem_info, 0, &red_cursor_cmd,
> to_physical(&cursor_cmd)));
> -    free(red_cursor_cmd.u.set.shape.data);
> -    free(cursor);
> +    g_free(red_cursor_cmd.u.set.shape.data);
> +    g_free(cursor);
>      memslot_info_destroy(&mem_info);
>  }
>  
> @@ -210,8 +210,8 @@ static void test_circular_empty_chunks(void)
>      }
>      g_test_assert_expected_messages();
>  
> -    free(cursor);
> -    free(chunks[0]);
> +    g_free(cursor);
> +    g_free(chunks[0]);
>      memslot_info_destroy(&mem_info);
>  }
>  
> @@ -252,8 +252,8 @@ static void test_circular_small_chunks(void)
>      }
>      g_test_assert_expected_messages();
>  
> -    free(cursor);
> -    free(chunks[0]);
> +    g_free(cursor);
> +    g_free(chunks[0]);
>      memslot_info_destroy(&mem_info);
>  }
>  
> diff --git a/server/tests/test-stat-file.c b/server/tests/test-stat-
> file.c
> index 901985a7..e6aef827 100644
> --- a/server/tests/test-stat-file.c
> +++ b/server/tests/test-stat-file.c
> @@ -38,7 +38,7 @@ static void stat_file(void)
>  
>      g_assert_nonnull(stat_file);
>      g_assert_nonnull(stat_file_get_shm_name(stat_file));
> -    filename = strdup(stat_file_get_shm_name(stat_file));
> +    filename = g_strdup(stat_file_get_shm_name(stat_file));
>      g_assert(access(filename, R_OK));
>  
>      /* fill all nodes */
> @@ -84,7 +84,7 @@ static void stat_file(void)
>      stat_file_unlink(stat_file);
>      g_assert_null(stat_file_get_shm_name(stat_file));
>      g_assert_cmpint(access(filename, F_OK),==,-1);
> -    free(filename);
> +    g_free(filename);
>  
>      stat_file_free(stat_file);
>  }
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]