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.

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

nodogsplash: update to version 5.0.2 #1023

Merged
merged 1 commit into from
Oct 10, 2023
Merged

nodogsplash: update to version 5.0.2 #1023

merged 1 commit into from
Oct 10, 2023

Conversation

mwarning
Copy link
Contributor

@mwarning mwarning commented Oct 9, 2023

Fix a buffer overflow in case the query string is too long.

Maintainer: me
Compile tested: Nexx wt3020, ramips/mt7620
Run tested: save as compiled; tested internet blocked and unblocked after splash site click

Fix a buffer overflow in case the
query string is too long.

Signed-off-by: Moritz Warning <moritzwarning@web.de>
@mwarning mwarning changed the title [draft] nodogsplash: update to version 5.0.2 nodogsplash: update to version 5.0.2 Oct 10, 2023
@mwarning mwarning merged commit fe1c67f into openwrt:master Oct 10, 2023
13 checks passed
@@ -40,6 +40,7 @@ define Package/nodogsplash/description
endef

define Package/nodogsplash/install
$(CP) ./files/* $(1)/
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uhh, this is not good. The PR was draft and then it was merged.
I would like to avoid this. You can not use CP for all. The previous solution was better, not the best.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So you want each file to be referenced?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants