Skip Menu |

This queue is for tickets about the Net-DHCP CPAN distribution.

Report information
The Basics
Id: 61693
Status: resolved
Priority: 0/
Queue: Net-DHCP
People
Owner: Nobody in particular
Requestors: dean [...] fragfest.com.au
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.66
Fixed in: (no value)

History Show all quoted text
  Mon Sep 27 09:36:11 2010 dean [...] fragfest.com.au - Ticket created
Subject: ->marshall() has large memory leak
the marshall() function is extremely useful to avoid repeatedly creating and destroying the Net::DHCP option. the same object can be used to decode subsequent packets. unfortunately the marshall function doesnt clear the previous data from addOptionsRaw(), so subsequent raw options are concatenated to the object gradually filling memory. if you create a new object each time, this isnt a problem.
  Mon Sep 27 09:44:16 2010 dean [...] fragfest.com.au - Correspondence added
this is solved easily enough by adding this code to the marhsall sub delete $self->{options} if $self->{options}; delete $self->{options_order} if $self->{options_order}; On Mon Sep 27 09:36:11 2010, djzort wrote: Show quoted text
> the marshall() function is extremely useful to avoid repeatedly creating > and destroying the Net::DHCP option. the same object can be used to > decode subsequent packets. > > unfortunately the marshall function doesnt clear the previous data from > addOptionsRaw(), so subsequent raw options are concatenated to the > object gradually filling memory. > > if you create a new object each time, this isnt a problem.
  Mon Sep 27 09:44:18 2010 dean [...] fragfest.com.au - Status changed from 'new' to 'open'
  Fri Nov 12 23:45:32 2010 dean [...] fragfest.com.au - Correspondence added
fixed in 0.67
  Fri Nov 12 23:45:33 2010 dean [...] fragfest.com.au - Status changed from 'open' to 'resolved'