Laser Visual segmentation fault

[osrf-migration]

Original report (archived issue) by François-Michel De Rainville (Bitbucket: fmder).

---

In the LaserVisual constructor of branch 1.9, the OnScan callback is registered before the first point is added to rayFan. When the computers slows down (in big simulations) this leads to a segmentation fault in the OnScan callback (the rayFan->SetPoint(0, ...) line).

In version 2.0, this has been fixed in the rewrite of the component.

Bringing the subscription of the callback on the last line solved the problem for me.

[osrf-migration]

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

---

• set assignee_account_id to "557058:5de38267-b118-494c-aa76-4fab35448816"
• set assignee to "scpeters (Bitbucket: scpeters, GitHub: scpeters)"

Thanks, I'll make a pull request.

[osrf-migration]

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

---

pull request #832

[osrf-migration]

Original comment by Steve Peters (Bitbucket: Steven Peters, GitHub: scpeters).

---

• changed state from "new" to "resolved"

pull request #832

[osrf-migration]

Original comment by amirhosein_shantia (Bitbucket: amirhosein_shantia).

---

Is this fix also applied to the ubuntu apt repo?
I am having the same segmentation fault problem while loading the kitche_dining model when I manually add collisions for the model using the same mesh from the data base.
Do I have to build gazebo 1.9 from source to fix this?

[osrf-migration]

Original comment by Jose Luis Rivero (Bitbucket: Jose Luis Rivero, GitHub: j-rivero).

---

The patch was added to gazebo 1.9 branch a bit after release 1.9.2 and it is present in our latest released version 1.9.3.

The osrf repository hosts the 1.9.3 version while the ROS repo still have the 1.9.2, so you would need to enable the osrf repository or just download and install the package manually.

[osrf-migration]

Original comment by amirhosein_shantia (Bitbucket: amirhosein_shantia).

---

Thanks, I got the 1.9.3 binary package from your link. It works great.

[osrf-migration]

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

---

• set version to "all"

[osrf-migration]

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).

---

• changed state from "resolved" to "closed"