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

RHEL 8.9 nullptr dereference #377

Closed
wants to merge 7 commits into from
Closed

RHEL 8.9 nullptr dereference #377

wants to merge 7 commits into from

Conversation

AlexScherba16
Copy link
Collaborator

  • changes in block_device in kernel
  • support new linux kernel function declarations
  • support for bd_has_submit_bio for block device

@st-matskevich st-matskevich changed the title Rhel8 9 nullptr dereference RHEL 8.9 nullptr dereference Jun 17, 2024
@Swistusmen
Copy link
Contributor

Is it all copied from main...Swistusmen:dattobd:supporting-ubuntu24 and #367 + changes made here: e9e8df1?

@AlexScherba16
Copy link
Collaborator Author

@Swistusmen without changes in tracer.

Swistusmen
Swistusmen reviewed Jun 20, 2024
View reviewed changes
src/blkdev.c
* Return:
* void.
*/
void dattobd_drop_super(struct super_block *sb)
Copy link
Contributor

Choose a reason for hiding this comment

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

Pls simplify to #if defined HAVE_GET_SUPER return drop_super(sb) #endif return;

Swistusmen
Swistusmen suggested changes Jun 20, 2024
View reviewed changes
Copy link
Contributor

@Swistusmen Swistusmen left a comment

Choose a reason for hiding this comment

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

Pls simplify dattobd_drop_super

@Swistusmen Swistusmen mentioned this pull request Jun 20, 2024
Open
@AlexScherba16 AlexScherba16 force-pushed the rhel8_9_nullptr_dereference branch 2 times, most recently from 5c3b7dc to f62a84b Compare June 27, 2024 19:28
@iamandrii
Copy link
Collaborator

Included to #382

@iamandrii iamandrii closed this Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Swistusmen Swistusmen Swistusmen requested changes

@st-matskevich st-matskevich st-matskevich approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AlexScherba16 @Swistusmen @iamandrii @st-matskevich