Closed
Bug 782271
Opened 12 years ago
Closed 9 years ago
signature generation doubles single quotes unnecessarily
Categories
(Socorro :: Backend, task)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: lars, Assigned: lars)
Details
in signatureUtilities.py (and signature_utilities.py), the base class will quote single quotes by doubling them. While this may be appropriate for Java signatures, it messes up C signatures.
Move the code that quotes single quotes into the Java sub-class rather than the base class.
Assignee | ||
Comment 1•11 years ago
|
||
I'm going through old issues and closing or resolving them. Is this still and issue that I ought to be addressing?
Flags: needinfo?(kairo)
Flags: needinfo?(benjamin)
Comment 2•11 years ago
|
||
I really don't know why we're doubling it in any case. I found example of doubling in both Java and C++ that looks "extra":
Java: https://crash-stats.mozilla.com/report/index/6fc7a0c7-c0a8-4baf-8e66-b7b002140225
C++: https://crash-stats.mozilla.com/report/index/451773cd-a5e9-4810-873e-f6ac32140225
(supersearch: https://crash-stats.mozilla.com/search/?signature=~%27&_facets=signature&_columns=date&_columns=signature&_columns=product&_columns=version&_columns=build_id&_columns=platform)
If I were guessing, I suspect the doubling comes from some very old SQL injection issue which doesn't apply now that we're using variable bindings properly.
Flags: needinfo?(kairo)
Flags: needinfo?(benjamin)
Assignee | ||
Comment 3•9 years ago
|
||
old and outdated - refile if this is still needed.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•