Anonymous | Login | Signup for a new account | 2024-12-05 04:26 CET |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0001026 | aMule | External Conn | public | 2006-12-13 21:27 | 2008-02-22 03:21 | ||||
Reporter | quinox | ||||||||
Assigned To | Xaignar | ||||||||
Priority | normal | Severity | crash | Reproducibility | always | ||||
Status | resolved | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | SVN | ||||||||
Target Version | Fixed in Version | SVN | |||||||
Summary | 0001026: aMuleD crash on a specific EC_OP_SEARCH_START packet | ||||||||
Description | It is possible to crash aMuled by sending a specific EC_OP_SEARCH_START packet. Still working on my own interface for aMule. Working with the protocol is nice and most things are logical, but aMule sure can't handle any unexpected problems: On transmitting the search packet, aMule will go down with the following message: amuled: ./libs/ec/ECTag.h:158: uint64_t CECTag::GetInt() const: Assertion `0' failed. Aborted This happens with CVS from 20061029 up to 20061211. | ||||||||
Additional Information | Hex dump of the evil packet 0000 00 00 00 20 00 00 00 51 ... ...Q 0008 26 00 4E 00 98 06 00 00 &.N..... 0010 00 06 71 75 65 72 79 00 ..query. 0018 00 9A 03 00 00 00 02 00 ........ 0020 00 00 9C 06 00 00 00 01 ........ 0028 00 00 9E 03 00 00 00 02 ........ 0030 00 00 00 A0 03 00 00 00 ........ 0038 02 00 00 00 A2 06 00 00 ........ 0040 00 06 6C 6F 63 61 6C 00 ..local. 0048 00 A4 06 00 00 00 01 00 ........ 0050 00 A6 03 00 00 00 02 00 ........ 0058 00 . | ||||||||
Tags | No tags attached. | ||||||||
Fixed in Revision | |||||||||
Operating System | Any | ||||||||
Attached Files | |||||||||
Notes | |
(0002190) Kry (manager) 2006-12-14 15:53 |
That's not an unhandled problem, it's a handled problem, actually :P That's why the assert is there. It's not a crash. |
(0002191) quinox (reporter) 2006-12-14 23:02 |
OK, but aMule shouldn't go down because of it should it? Just ignoring the packet should be fine, perhaps with sending back an error |
(0002192) Kry (manager) 2006-12-16 16:54 |
don't compile on debug mode if you don't want debug behaviour? |
(0002198) quinox (reporter) 2006-12-17 19:31 |
Unless CVS code has some debug turned on by default, I'm not: **** aMule Core **** Prefix where aMule should be installed? /home/quinox/chroot/ Should aMule be compiled with i18n support? yes Should aMule be compiled in debug mode? no Should aMule be compiled with profiling? no Should aMule be compiled with optimizations? no Should aMule monolithic application be built? yes Should aMule daemon version be built? yes Should aMule remote gui be built? (EXPERIMENTAL) yes |
(0002200) Kry (manager) 2006-12-17 21:05 |
You have wx compiled with debug. |
(0002201) quinox (reporter) 2006-12-18 16:54 |
Right, I'll look into that - I guess the problem isn't with aMule then, feel free to close this bug :) |
(0002202) Kry (manager) 2006-12-18 21:49 |
Nah... I'm leaving this around. Maybe some logging is betetr than an assert, but then again some hard debug is always welcome... I'll think about it. |
(0002648) Xaignar (manager) 2008-02-22 03:21 |
This situation has been improved in current SVN, where wxASSERT is now used instead of assert in EC code, allowing for aMule to continue running after an asserted has been triggered. |
Issue History | |||
Date Modified | Username | Field | Change |
2006-12-13 21:27 | quinox | New Issue | |
2006-12-13 21:27 | quinox | Operating System | => Any |
2006-12-14 15:53 | Kry | Note Added: 0002190 | |
2006-12-14 23:02 | quinox | Note Added: 0002191 | |
2006-12-16 16:54 | Kry | Note Added: 0002192 | |
2006-12-17 19:31 | quinox | Note Added: 0002198 | |
2006-12-17 21:05 | Kry | Note Added: 0002200 | |
2006-12-18 16:54 | quinox | Note Added: 0002201 | |
2006-12-18 21:49 | Kry | Note Added: 0002202 | |
2008-02-22 03:21 | Xaignar | Status | new => resolved |
2008-02-22 03:21 | Xaignar | Fixed in Version | => SVN |
2008-02-22 03:21 | Xaignar | Resolution | open => fixed |
2008-02-22 03:21 | Xaignar | Assigned To | => Xaignar |
2008-02-22 03:21 | Xaignar | Note Added: 0002648 |
Copyright © 2000 - 2024 MantisBT Team |