ETSI's Bug Tracker - GeoNetworking | |||||
| View Issue Details | |||||
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0005846 | GeoNetworking | Base Spec | public | 08-12-2010 13:00 | 20-02-2015 14:39 |
| Reporter | Andras Kovacs | ||||
| Assigned To | Andreas Festag | ||||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Platform | OS | OS Version | |||
| Product Version | TS 102 636 4-1 V0.0.9 | ||||
| Target Version | Fixed in Version | Next Release | |||
| Summary | 0005846: TS102636-4-1 v0.0.9 / Annex D.3: pseudo-code is misleading | ||||
| Description | While the intention of the algorithm is understandable from the textual description, the following part of pseudo-code is very misleading: IF (DEST_LL_ADDR = L_LL_ADDR) THEN ADD FP TO B Set TO = MAX Set COUNTER = 1 ELSE Add FP in B Set TO Set COUNTER = 0 ENDIF Start TIMER(TO) Return 0 Specific issues: - the role of COUNTER (later its comparison against MAX_COUNTER) are not described. Since the GeoBroadcast handling procedure already involves packet duplicate detection, it effectively checks against COUNTER limit value 1. So this variable is not needed here. (same issue with state diagram) - the line with 'Set TO' seems incomplete. - L_LL_ADDR is not defined (probably it's the local link layer address of the ITS station) - the logic appears to be wrong, as (DEST_LL_ADDR = L_LL_ADDR) condition is meant to indicate immediate re-broadcast | ||||
| Steps To Reproduce | |||||
| Additional Information | Perhaps rather than fixing the pseudo-code and state diagram, they should be just removed now. It would save effort to add them back in a later version, which would describe the recommended iterative re-broadcast timer algorithm. | ||||
| Tags | No tags attached. | ||||
| Relationships | |||||
| Attached Files | |||||
| Issue History | |||||
| Date Modified | Username | Field | Change | ||
| 08-12-2010 13:00 | Andras Kovacs | New Issue | |||
| 01-02-2011 05:58 | Andreas Festag | Status | new => resolved | ||
| 01-02-2011 05:58 | Andreas Festag | Fixed in Version | => Next Release | ||
| 01-02-2011 05:58 | Andreas Festag | Resolution | open => fixed | ||
| 01-02-2011 05:58 | Andreas Festag | Assigned To | => Andreas Festag | ||
| 01-02-2011 05:58 | Andreas Festag | Note Added: 0009993 | |||
| 20-02-2015 14:39 | Sebastian Muellers | Note Added: 0012794 | |||
| 20-02-2015 14:39 | Sebastian Muellers | Status | resolved => closed | ||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||