x

Search in
Sort by:

Question Status:

Search help

  • Simple searches use one or more words. Separate the words with spaces (cat dog) to search cat,dog or both. Separate the words with plus signs (cat +dog) to search for items that may contain cat but must contain dog.
  • You can further refine your search on the search results page, where you can search by keywords, author, topic. These can be combined with each other. Examples
    • cat dog --matches anything with cat,dog or both
    • cat +dog --searches for cat +dog where dog is a mandatory term
    • cat -dog -- searches for cat excluding any result containing dog
    • [cats] —will restrict your search to results with topic named "cats"
    • [cats] [dogs] —will restrict your search to results with both topics, "cats", and "dogs"

TQueue> results in Memory Leak / corruption

From \Engine\Source\Runtime\Core\Public\Containers\Queue.h

    ~TQueue()
     {
         while (Tail != nullptr)
         {
             TNode* Node = Tail;
             Tail = Tail->NextNode;
 
             delete Node;
         }
     }

The destructors condition functions purely off the assumption that the tail node is properly initialized, and does not account for any number of scenarios where tail's next node is not valid but is also not c++ 11 null constant.

If tqueue or any nodes are initialized under sketchy conditions, not properly initialized, or not initialized at all, the destructor can go on a memory corrupting rampage since there is no validation, no type checking, no bounds checking, or any kind of cross check...

Product Version: UE 4.8
Tags:
more ▼

asked Jun 28 '15 at 02:10 AM in Bug Reports

avatar image

BunniPhD
136 4 68 15

avatar image Doug E ♦♦ STAFF Jun 29 '15 at 05:46 PM

Hey BunniPhD-

Do you have the log files or callstack from a crash caused by this memory leak? Are there steps I can follow to reproduce the crash in a new project on my end?

Cheers

Doug Wilson

(comments are locked)
10|2000 characters needed characters left
Viewable by all users

1 answer: sort voted first

Hey BunniPhD-

We've not heard from you in a few days and for tracking purposes I will be marking this post as resolved. If you are still having issues with the TQueue causing memory leaks feel free to respond to this post to reopen it for further investigation. Please be sure to include the information requested in my previous comment to help me reproduce the issue locally.

Cheers

Doug Wilson

more ▼

answered Jul 09 '15 at 12:59 PM

(comments are locked)
10|2000 characters needed characters left
Viewable by all users
Your answer
toggle preview:

Up to 5 attachments (including images) can be used with a maximum of 5.2 MB each and 5.2 MB total.

Follow this question

Once you sign in you will be able to subscribe for any updates here

Answers to this question