Ticket #223 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Local Variable old_f not var scoped

Reported by: tobylaroni Owned by: sean@…
Type: defect Priority: normal
Milestone: Component: Miscellaneous
Version: Severity: normal
Keywords: Cc:

Description

Couldn't find this as an existing ticket, so wasn't sure if it was on the radar or not.

In fuseboxDoFuseaction.cfc, in the compile method, the old_f variable isn't var scoped at the top of the method.

You have a var scoped old_p, which isn't used in the remainder of the method -- is this supposed to be old_f instead.

Not sure whether this is at all important, but it could, theoretically, lead to memory leaks, race conditions.

By the way, I didn't stumble upon this through intelligence, sheer force, or even luck. I used Mike Schierberl's varscoper tool.

Attachments

Change History

Changed 2 years ago by scorfield

  • status changed from new to closed
  • resolution set to fixed

(In [449]) Fixes #223 by correcting the local variable old_p to be old_f.

Changed 2 years ago by scorfield

(In [450]) Addresses #223 by merging fix from trunk.

Note: See TracTickets for help on using tickets.