Bug 17465 - Double quotes aren't being escaped in text fields
: Double quotes aren't being escaped in text fields
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.6.0
: PC Windows XP
: P3 normal (vote)
: 7.6.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-18 15:34 UTC by Jim McAtee
Modified: 2011-09-02 08:37 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jim McAtee 2011-08-18 15:34:59 UTC
Double quotes in a string value displayed in a text field in the web interface causes the text to appear truncated. They need to be escaped when being output in a form field.
Comment 1 Michael Herger 2011-08-18 23:30:45 UTC
More info from the forums:

"It looks like the search string is directly passed to the SQL query:
searching for "don_t" will bring you "don't", searching for "%" will
bring you anything (as does a single "*", by the way - not sure it's
part of the SQL standard)."
Comment 2 Jim McAtee 2011-08-19 02:53:33 UTC
I think you may have gotten your bugs mixed up. That comment has nothing to do with this bug.
Comment 3 Michael Herger 2011-09-02 05:06:04 UTC
Jim - I can reproduce this in the case of the search. Do you see this happen in other places too?
Comment 4 SVN Bot 2011-09-02 05:20:53 UTC
 == Auto-comment from SVN commit #33293 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=33293 ==

Fixed Bug: 17465
Description: html escape input field values or double quotes would break them
Comment 5 Jim McAtee 2011-09-02 08:37:10 UTC
(In reply to comment #3)
> Jim - I can reproduce this in the case of the search. Do you see this happen in
> other places too?

I was talking about anywhere within the web UI, including Settings. I see you've  fixed a couple of the settings pages.