Skip to content
Permalink
Browse files Browse the repository at this point in the history
Merge pull request from GHSA-jjq6-mh2h-g39h
  • Loading branch information
rousskov committed Mar 16, 2021
1 parent 2bde9c7 commit 0518249
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/http/RegisteredHeaders.cc
Expand Up @@ -37,7 +37,7 @@ HeaderTableRecord::HeaderTableRecord(const char *n, HdrType theId, HdrFieldType
const HeaderTableRecord&
HeaderLookupTable_t::lookup (const char *buf, const std::size_t len) const {
const HeaderTableRecord *r = HttpHeaderHashTable::lookup(buf, len);
if (!r)
if (!r || r->id == Http::HdrType::OTHER)
return BadHdr;
return *r;
}
Expand Down

2 comments on commit 0518249

@rousskov
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a GutHub UX bug that creates commits with wrong commit messages. We are now aware of this GitHub problem and will do our best to work around it in the future. Ignore this commit for now.

@yadij
Copy link
Contributor

@yadij yadij commented on 0518249 Mar 19, 2021

Choose a reason for hiding this comment

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

@utkarsh2102 I think it safe to say NO it is not that CVE.

I hope you do not mind: To avoid any future search engine confusion with that CVE I am deleting your comment.

Please sign in to comment.