Site maintenance: Sun 15 Nov, from 12:00 UTC, about 4–6 hours. Details

This cutover relies on some testing that is still incomplete so may be cancelled at short notice

Forums / Upgrading from 1.3.x to 1.3.9 / Rename Admin folder security patch v1.3.8

Rename Admin folder security patch v1.3.8

Locked

Started by alimtlai on 24 Aug 2009, 11:46 AM · Views: 7,415

Results 1 to 19 of 19
This thread is locked. New replies are disabled.
24 Aug 2009, 11:46 AM
#1
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Rename Admin folder security patch v1.3.8

After installed the security patch, I can login but cannot update anything inside the admin configuration panel. It just jumps out to the main admin panel without any error message. I double checked that I followed exactly the instruction. Anyone got an idea why? I'm in a hurry to fix this. Thanks a lot.

24 Aug 2009, 11:48 AM
#2
kuroi avatar

kuroi

Totally Zenned

Join Date:
Apr 2006
Location:
London, UK
Posts:
10,475
Plugin Contributions:
11

Re: Rename Admin folder security patch v1.3.8

This most likely means that you haven't managed to properly over-write the admin/includes/functions/html_output.php file with the new version.

Kuroi Web Design and Development | Twitter

(Questions answered in the forum only - so that any forum member can benefit - not by personal message)

24 Aug 2009, 11:56 AM
#3
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

No, I've updated the file and changed its mode to 675 as original already. But still got this problem.

24 Aug 2009, 5:36 PM
#4
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

One more error was found. I even can't switch the currency for the "product detail" page. It will routed to the "page not found" page. It is really critical and I need to fix it urgently, otherwise, I have to rollback the security patch. Could anyone help?

24 Aug 2009, 5:38 PM
#5
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

How many files did you change/add?
Which folders did you put them in?

Sounds like you put the files in the wrong places.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

25 Aug 2009, 2:55 AM
#6
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

I've put in the new files and amended two files.

  1. includes/function/html_output.php
  2. renamed admin folder/includes/configure.php

I double checked several times and pretty sure I follow exactly the instruction. But then I have these two problems:

  1. You can't switch the currency in product detail page. It will jump to "page not found" page.
  2. I can't update any configuration in admin panel. It jump out to the main admin panel without any error message.

Don't you guys having the same problem?

25 Aug 2009, 3:15 AM
#7
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

As a suppliment, the url link for page not found is somewhat like this

/page_not_found/securityToken/16fb69b84332cf65674.....

25 Aug 2009, 3:24 AM
#8
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

You touched the wrong html_output.php file.
The documentation tells you to edit/update the /admin/includes/functions/html_output.php file.
Also, the zip includes a folder structure that puts that updated file in the appropriate place under the admin folder structure, not in the non-admin area.

You'll need to put back the original non-admin html_output.php file, and then edit/patch the admin one.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

25 Aug 2009, 3:51 AM
#9
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

OH.....I'm so silly. Yes, you are right, I edited the wrong html_output.php. I've fixed this file but it can only solve the configuration problem. The currency problem still persist. You can visit my testing site to see what's happening.

http://www.popartonline.com/zencart2/andy-warhol-remake-animations-pop-art-c-130_131/bat-grapnel-gun-for-batman-p-2931

I double checked all the new files are put in the admin folder already. No mistake for this part.

25 Aug 2009, 3:56 AM
#10
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

Looks like something wrong with your so-called SEO addon.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

25 Aug 2009, 3:59 AM
#11
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

But I don't have this problem before installing the security patch. Are you suggesting there is some conflict between this security patch and the SEO add-on? Anyway to handle this? Remove the SEO addon? Any workaround solution is also fine.

25 Aug 2009, 4:03 AM
#12
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

I suspect your problem is that you have an SEO addon that came with an altered version of /includes/functions/html_output.php and when you mistakenly edited that file you ended up damaging it, and when you tried to reverse it you didn't put back what was there before editing, and likely put back an old copy from before adding the SEO addon.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

25 Aug 2009, 4:17 AM
#13
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

No, I renamed the original file in the same folder, so I simply rename it back. I double checked the content, it was the one before editing. Any other hints?

25 Aug 2009, 4:26 AM
#14
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

If you turn off your SEO addon, does the currency box work properly?
Did your site's main .htaccess file get altered recently? Maybe something's broken in there.
If turning off the SEO addon "fixes" the problem, then you'll have to work with that addon's support/discussion about how to re-enable it.

Keep in mind that the June 2009 security patch ONLY touches admin-related files, and thus has NO effect on any of the files in your storefront. So, if your storefront is "suddenly" broken, then it's not the fault of the patch ... so I'd start looking at user-error, or maybe files getting hacked and not properly cleaned up, or other contributing factors.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.

25 Aug 2009, 4:53 AM
#15
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

How to turn off SEO? I'm using Simple SEO URL. Seems that there is no such option to turn off.

I double checked the current content with the one from simple SEO URL, it was the same. So I confirm my rollback is successful.

I also notice that you are one of member contributing to the amendment of this html_output.php. Really feel honored to have you helping me on this issue. Thanks a lot.

I confirm that the problem only appear after the security patch installation, you can see this website, it is the one before installating the security patch.

http://www.smarthost4u.org/sclients/fineartmegastore/

25 Aug 2009, 7:03 AM
#16
yellow1912 avatar

yellow1912

Totally Zenned

Join Date:
Oct 2006
Posts:
5,422
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

DrByte is not just a member, he is a core developer :)

SSU can be turned off via admin->configuration->ssu setting.

SSU does edit your html_output.php, and this edit does not conflict with any zencart security patch. There should be absolutely no problem if you applied the patch correctly.

I suggest you recheck the html_output.php goes inside the SSU package and compare with the current one.

I no longer provide installation support on forum for all my modules. However, if there are real bugs with the modules please feel free to contact me

25 Aug 2009, 7:46 AM
#17
alimtlai avatar

alimtlai

New Zenner

Join Date:
Sep 2008
Posts:
87
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

After turning SSU off, the currency switching has no problem. I compared the below file with the one in simple SEO URL, the only difference is the security patch.

admin/includes/function/html_output.php

Are you suggesting simple SEO URL has conflict with this security patch? Any hints? How about use another SEO URL addon?

25 Aug 2009, 4:40 PM
#18
yellow1912 avatar

yellow1912

Totally Zenned

Join Date:
Oct 2006
Posts:
5,422
Plugin Contributions:
0

Re: Rename Admin folder security patch v1.3.8

alimtlai:

After turning SSU off, the currency switching has no problem. I compared the below file with the one in simple SEO URL, the only difference is the security patch.

admin/includes/function/html_output.php

Are you suggesting simple SEO URL has conflict with this security patch? Any hints? How about use another SEO URL addon?
No, it cannot have any conflict because

  1. logically, this patch does not affect ssu in anyway (and vice versa)
  2. we are using this patch along with ssu for our own site and clients sites.

I suspect in the process of applying the patch you messed up something else :)

I no longer provide installation support on forum for all my modules. However, if there are real bugs with the modules please feel free to contact me

25 Aug 2009, 6:08 PM
#19
drbyte avatar

drbyte

Sensei

Join Date:
Jan 2004
Posts:
63,511
Plugin Contributions:
176

Re: Rename Admin folder security patch v1.3.8

alimtlai:

After turning SSU off, the currency switching has no problem. I compared the below file with the one in simple SEO URL, the only difference is the security patch.

admin/includes/function/html_output.php

Are you suggesting simple SEO URL has conflict with this security patch? Any hints? How about use another SEO URL addon?
It still seems like maybe your non-admin version of html_output.php is not the right one.

.
Zen Cart - putting the dream of business ownership within reach of anyone!
Donate to: DrByte directly or to the Zen Cart team as a whole

Remember: Any code suggestions you see here are merely suggestions. You assume full responsibility for your use of any such suggestions, including any impact ANY alterations you make to your site may have on your PCI compliance.
Furthermore, any advice you see here about PCI matters is merely an opinion, and should not be relied upon as "official". Official PCI information should be obtained from the PCI Security Council directly or from one of their authorized Assessors.