Ticket #1900 (closed enhancement: wontfix)

Opened 9 years ago

Last modified 8 years ago

replacing-free-by-g_free

Reported by: vit_r Owned by: vit_r
Priority: trivial Milestone:
Component: mc-core Version:
Keywords: Cc:
Blocked By: Blocking:
Branch state: Votes for changeset:

Description

Look patchname

Attachments

1900-replacing-free-by-g_free.patch (8.5 KB) - added by vit_r 9 years ago.

Change History

Changed 9 years ago by vit_r

comment:1 follow-up: ↓ 3 Changed 9 years ago by vit_r

( src/util.h: #define MC_PTR_FREE(ptr) do { g_free(ptr); (ptr) = NULL; } while (0) )

Suggestion:

Lets replace all g_free by MC_PTR_FREE

comment:2 Changed 9 years ago by andrew_b

Please, don't fill the "Assign to:" field when you create a new ticket.

Don't fix the Samba codebase -- it will be removed soon.

Lets replace all g_free by MC_PTR_FREE

Often no need to set pointer to NULL after memory deallocation.

comment:3 in reply to: ↑ 1 Changed 9 years ago by slyfox

Replying to vit_r:

Lets replace all g_free by MC_PTR_FREE

Please, don't.

comment:4 Changed 8 years ago by andrew_b

  • Status changed from new to closed
  • Version version not selected deleted
  • Resolution set to wontfix
  • Milestone 4.7 deleted
Note: See TracTickets for help on using tickets.