View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000822 | Ecere SDK | ide:debugger | public | 2013-04-24 04:24 | 2014-05-14 11:14 |
Reporter | redj | Assigned To | jerome | ||
Priority | immediate | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | no change required | ||
Product Version | 0.44.07 | ||||
Target Version | 0.44.10 64 | ||||
Summary | 0000822: GDB looking where we don't want it to look in strings | ||||
Description | GDB looking where we don't want it to look in strings: - Past the null character (That *could* be OK on fixed size arrays, but definitely not on pointers) - Repeats in the middle of a string: 0x2ebecb8 \"http://demo.opengeo.org/geoserver/wms?service=wms&version=1.1.1&request=GetMap&styles=&srs=EPSG%3A4326&format=image%2Fpng&transparent=true&layers=za:za_vegetation&bbox=30.\", '0' <repeats 13 times>, \",-33.2812500000000,30.4687500000000,-32.8125000000000&width=256&height=256\" | ||||
Tags | No tags attached. | ||||
|
We are requesting from GDB more memory than was allocated, but it does not seem to be causing any issue. The code was recently improved to query starting with 256 chars at a time, up to a maximum of 4096. https://github.com/ecere/ecere-sdk/commit/fa67a6c6dbbfc7481e3af0925ad1bc5812b16f74 Reading the data this way has the advantage of controlling the maximum number of characters being read. It also avoids having to process escape codes. It was possibly originally changed to this way so that expressions GDB do not understand work properly. Those would otherwise require an evaluation of an address casted to an unsigned char * (and then proper escaping processing), but would still have the issue of the maximum characters if there was some large buffer did not have nul characters in it. |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-04-24 04:24 | redj | New Issue | |
2013-04-24 20:19 | jerome | Priority | normal => immediate |
2014-05-13 23:59 | jerome | Assigned To | => jerome |
2014-05-13 23:59 | jerome | Status | new => assigned |
2014-05-13 23:59 | jerome | Target Version | 0.44.12 Fixes => 0.44.10 64 |
2014-05-14 11:14 | jerome | Status | assigned => closed |
2014-05-14 11:14 | jerome | Note Added: 0001293 | |
2014-05-14 11:14 | jerome | Resolution | open => no change required |