Last Comment Bug 785320 - [b2g-bluetooth] Crash after renaming device in Settings/Bluetooth
: [b2g-bluetooth] Crash after renaming device in Settings/Bluetooth
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: DOM: Device Interfaces (show other bugs)
: unspecified
: ARM Gonk (Firefox OS)
: -- normal (vote)
: mozilla17
Assigned To: Eric Chou [:ericchou] [:echou]
:
Mentors:
Depends on:
Blocks: b2g-bluetooth
  Show dependency treegraph
 
Reported: 2012-08-24 00:10 PDT by Eric Chou [:ericchou] [:echou]
Modified: 2012-08-25 08:46 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
v1: Fix crash after renaming (867 bytes, patch)
2012-08-24 03:25 PDT, Eric Chou [:ericchou] [:echou]
kyle: review+
Details | Diff | Review

Description Eric Chou [:ericchou] [:echou] 2012-08-24 00:10:14 PDT
There's a "Rename Device" button for user to rename this device. Once you've done editing and click 'ok' at "Change Phone Name" page, it'll crash.
Comment 1 Eric Chou [:ericchou] [:echou] 2012-08-24 03:25:18 PDT
Created attachment 654969 [details] [diff] [review]
v1: Fix crash after renaming

When calling SetProperty(), a pointer to pointer should be passed in.
Comment 2 Kyle Machulis [:kmachulis] [:qdot] 2012-08-24 19:40:41 PDT
I've got this into try at https://tbpl.mozilla.org/?tree=Try&rev=6628dd305e39, will land it myself if that goes through so we can get https://github.com/mozilla-b2g/gaia/issues/3429 unblocked
Comment 3 Kyle Machulis [:kmachulis] [:qdot] 2012-08-24 20:32:55 PDT
https://hg.mozilla.org/integration/mozilla-inbound/rev/86b8965a4c7f
Comment 4 Ryan VanderMeulen [:RyanVM] 2012-08-25 08:46:15 PDT
https://hg.mozilla.org/mozilla-central/rev/86b8965a4c7f

Note You need to log in before you can comment on or make changes to this bug.