- Status Closed
- Percent Complete
- Task Type Bugs
- Category Plugins
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by daurnimator - 2009-06-24
Last edited by mcuelenaere - 2009-06-24
Opened by daurnimator - 2009-06-24
Last edited by mcuelenaere - 2009-06-24
FS#10378 - lua causes segfalts when loading a nonexisting file
Make a lua script that trys to load a non existing file, eg:
loadfile ( “/file” )
OR
dofile ( rb.current_path ( ) .. “randomfile” )
run it; A segmentation fault occurs. (At least in my sim)
I’ve tried to narrow it down, and it seems to be causes at line 557 of apps/plugins/lua/lauxlib.c
“const char *filename = lua_tostring(L, fnameindex) + 1” =⇒ if you take away the +1, it works, but I think that would change what its pointing too….
Closed by mcuelenaere
2009-06-24 10:39
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-06-24 10:39
Reason for closing: Fixed
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
In r21488.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
That code isn’t ours, so I think this should also get reported upstream.
edit:
This seems like it’s a mistake in my code, I’ll look into it