View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003346 | 3 - Current Dev List | Bug | public | 2019-06-11 08:35 | 2019-06-15 22:46 |
| Reporter | K7ZCZ | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | new | Resolution | open | ||
| Product Version | |||||
| Target Version | Fixed in Version | ||||
| Summary | 0003346: LicenseValidator's ReadProductProperties method catches all exceptions | ||||
| Description |
catch (...)
{
//errorMessage = ex.Message;
}
It's an anti-pattern to catch all exceptions in most cases. Here, for sure -- no cleanup is performed, and the one chance at error reporting is commented-out (because it wouldn't work -- no such variables here). This code may pass a NULL pointer to OutputDebugString(), and the catch(...) code will make detecting and debugging that occurrence difficult. | ||||
| Tags | No tags attached. | ||||
| Module | SW License Mgmt | ||||
| Sub-Module | SW License Client | ||||
| Testing | Not Started | ||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2019-06-11 08:35 | K7ZCZ | New Issue | |
| 2019-06-15 10:43 | WA9PIE | Description Updated | View Revisions |
| 2019-06-15 10:43 | WA9PIE | Module | QLM => SW License Mgmt |
| 2019-06-15 10:43 | WA9PIE | Sub-Module | QLM Client => SW License Client |
