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"

[Bug Report] Null pointer crash in UProceduralMeshComponent

In 4.10 it is possible to get a null "ProcSceneMeshProxy" inside UProceduralMeshComponent::UpdateMeshSection() and also UProceduralMeshComponent::SetMeshSectionVisible().

Inside the "ENQUEUE_UNIQUE_RENDER_COMMAND...", there should be a check for null.

e.g.

                 if (ProcMeshSceneProxy) //New null check needed
                 {
                     ProcMeshSceneProxy->SetSectionVisibility_RenderThread(SectionIndex, bNewVisibility);
                 }

Think this can happen when setting the actor to hidden that the procedural mesh is attached to. Not every time though, probably race condition between game & render thread.

Product Version: UE 4.10
Tags:
more ▼

asked May 27 '16 at 10:11 PM in Bug Reports

avatar image

scamp
29 4 4 7

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

1 answer: sort voted first

Hello scamp,

Thank you for reporting this. Have you considered creating a Pull Request for this change on our GitHub repository? If you do so, a developer related to this field will review the code change directly to ensure that it fits with our standards and doesn't affect anything else. If the request is accepted, you'll become an Engine Contributor as well.

If you would rather not make a Pull Request however, please let me know and I'll place a bug report in for you.

more ▼

answered May 31 '16 at 03:36 PM

avatar image scamp May 31 '16 at 07:01 PM

I'll create the pull request, wasn't sure the best means of reporting these things. Thanks!

avatar image Matthew J May 31 '16 at 07:19 PM

No problem. Posting here and having us do a bug report is always best when you don't know of how to fix the problem yourself, but pull requests are better when you have an idea that we can work off of. It usually ends up being a quicker process to get through as well, depending on how important it is, along with being able to have direct contact with the person looking into the issue if they have any questions.

If you don't mind, could you link the pull request here whenever you enter it so that anyone who comes across this can check on the progress?

avatar image scamp May 31 '16 at 10:02 PM

Pull request Hopefully created it on the correct branch. Took longer than expected to figure it all out. Let me know if anything's wrong.

avatar image Matthew J Jun 01 '16 at 01:11 PM

Everything looks fine other than the branch that you've pushed it to. For changes to the engine, the pull request needs to be pushed to the Master branch, as the Release branch is something that is branch off from Master at a certain point and things are selectively added to it.

avatar image scamp Jun 01 '16 at 03:43 PM

Ok yeah makes sense. Think the doc I was referencing may be out of date. Link. (is this trello board run by a 3rd party perhaps? Don't see anything epic specific looking at it now)

Turns out someone else submitted the fix a few days earlier (here) so I've closed mine. Next time should go a bit smoother. Thanks for the help!

avatar image Matthew J Jun 01 '16 at 04:44 PM

That Trello board is run by a third party. If I remember correctly, the only thing we officially have on Trello is the UE4 Roadmap. I'm glad to hear that it's all sorted out. Looks like a dev is already on the case as well.

(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