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

util/mime: fix integer warnings #7074

Merged
merged 1 commit into from
Mar 3, 2022

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/4516

Describe changes:

  • Fix integer warnings -Wimplicit-int-conversion in util-decode-mime.c

It does not fix the tolower stuff taken care of by #7073

Part of #7006 modified : FindBuffer now takes an uint16
The only problematic part is about extract_urls_schemes which uses strlen on strings coming from configuration.
This length is now checked when reading the configuration in SMTPConfigure
The other uses of FindBuffer use fixed strings (whose length fits in a u8) or strings whose length is checked cf BOUNDARY_BUF

@catenacyber catenacyber requested a review from a team as a code owner February 28, 2022 08:10
@codecov
Copy link

codecov bot commented Feb 28, 2022

Codecov Report

Merging #7074 (ae6c416) into master (ec01a94) will increase coverage by 2.62%.
The diff coverage is 72.00%.

@@            Coverage Diff             @@
##           master    #7074      +/-   ##
==========================================
+ Coverage   75.54%   78.16%   +2.62%     
==========================================
  Files         628      628              
  Lines      185325   185466     +141     
==========================================
+ Hits       139996   144972    +4976     
+ Misses      45329    40494    -4835     
Flag Coverage Δ
fuzzcorpus 60.01% <37.50%> (+5.29%) ⬆️
suricata-verify 54.62% <60.86%> (+0.01%) ⬆️
unittests 63.17% <47.61%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@victorjulien victorjulien mentioned this pull request Mar 2, 2022
@victorjulien victorjulien merged commit ae6c416 into OISF:master Mar 3, 2022
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 28, 2025
Expose base64 standard, and standard_no_pad encoding to Lua scripts.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 28, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 28, 2025
Create a lenient base64 decode for Lua that is indifferent to spacing
and allows trailing data.

Expose to Lua as "decode_lenient".

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
Expose base64 standard, and standard_no_pad encoding to Lua scripts.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Jan 30, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 10, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 10, 2025
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 13, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
jasonish added a commit to jasonish/suricata that referenced this pull request Feb 13, 2025
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 13, 2025
Export our base64 decoding and encoding functions to Lua.

Ticket: OISF#7074
victorjulien pushed a commit to victorjulien/suricata that referenced this pull request Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants