Multiple vulnerabilities have been discovered in OpenX, which can be exploited by malicious people to conduct cross-site scripting, cross-site request forgery, and file inclusion attacks and by malicious users to conduct script insertion and SQL injection attacks.

1) Input passed to the "clientid" parameter in "www/admin/banner-acl.php", "www/admin/banner-edit.php", "www/admin/campaign-zone.php", "www/admin/advertiser-campaigns.php", "www/admin/campaign-banners.php", and "www/admin/banner-activate.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

2) Input passed to the "orderdirection" and "listorder" parameters in "www/admin/userlog-index.php" and "www/admin/stats.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

3) Input passed to the "origPublisherId" parameter in "www/admin/userlog-index.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

4) Input passed to "setPerPage", "day", "period_end", "period_start", and "statsBreakdown" parameters in "www/admin/stats.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

5) Input passed to the "campaignid" parameter in "www/admin/banner-acl.php", "www/admin/banner-edit.php", "www/admin/banner-acl.php", "www/admin/campaign-zone.php", and "www/admin/campaign-banners.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

6) Input passed to the "bannerid" parameter in "www/admin/banner-acl.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

7) Input passed to the "affiliateid" parameter in "www/admin/zone-probability.php", "www/admin/zone-invocation.php", "www/admin/affiliate-zones.php", and "www/admin/zone-include.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

8) Input passed to the "zoneid" parameter in "www/admin/zone-probability.php", "www/admin/zone-invocation.php", and "www/admin/zone-include.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

9) Input passed to the "userid" parameter in "www/admin/admin-user.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

10) Input passed to the "thirdpartytrack" parameter in "www/admin/admin-generate.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

11) Input passed to the "agencyid" parameter in "www/admin/agency-edit.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

12) Input passed to the "codetype" parameter in "www/admin/affiliate-preview.php" is not properly sanitised before being returned to the user. This can be exploited to execute arbitrary HTML and script code in a user's browser session in the context of an affected site.

13) The application allows users to perform certain actions via HTTP requests without performing any validity checks to verify the requests. This can be exploited to e.g. perform script insertion attacks via the "timezone" parameter in www/admin/account-preferences-timezone.php by tricking the user into visiting a malicious web site.

14) Input passed to the "name" and "description" parameters in "www/admin/channel-edit.php" is not properly sanitised before being used. This can be exploited to insert arbitrary HTML and script code, which is executed in a user's browser session in the context of an affected site when the malicious entry is viewed.

15) Input passed to the "campaignid" parameter in "www/admin/banner-acl.php", "www/admin/campaign-edit.php", and "www/admin/banner-edit.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

16) Input passed to the "bannerid" parameter in "www/admin/banner-acl.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

17) Input passed to the "listorder" parameter in "www/admin/userlog-index.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

18) Input passed to the "affiliateid" parameter in "www/admin/zone-probability.php", "www/admin/channel-edit.php", "www/admin/zone-invocation.php", and "www/admin/zone-include.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

19) Input passed to the "clientid" parameter in "www/admin/campaign-banners.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

20) Input passed to the "zoneid" parameter in "www/admin/zone-delete.php" and "www/admin/zone-include.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

21) Input passed to the "channelid" parameter in "www/admin/channel-acl.php" is not properly sanitised before being used in SQL queries. This can be exploited to manipulate SQL queries by injecting arbitrary SQL code.

22) Input passed to the "MAX_type" parameter in "www/delivery/fc.php" and to the "lang" parameter in "www/admin/numberFormat.js.php" is not properly verified before being used to include files. This can be exploited to include arbitrary files from local resources via directory traversal attacks.

NOTE: Other vulnerabilities may also exist.

The vulnerabilities are confirmed in version 2.6.3. Other versions may also be affected.

Solution
Update to version 2.4.10 or 2.6.4, which fixes some of the vulnerabilities. Vulnerabilities #9 and #11 remain unfixed.
Further details available to Secunia VIM customers

Provided and/or discovered by
Sarid Harper, Secunia.

The local file inclusion vulnerability via the "MAX_type" parameter (#22) was also independently discovered and publicly reported by Charlie.

Changelog
Further details available to Secunia VIM customers

Original Advisory
Secunia Research:
http://secunia.com/secunia_research/2009-4/

Charlie:
http://milw0rm.com/exploits/7883

Deep Links
Links available to Secunia VIM customers