Opened 43 hours ago

#1952 new defect

Outdated data-out assertion?

Reported by: Eaton Zveare Owned by:
Priority: minor Milestone: undecided
Component: smartctl Version: 7.5
Keywords: Cc:

Description

Hi, I noticed when sending an ATA data-out command, this assertion blocks commands that have multiple sectors: https://www.smartmontools.org/browser/smartmontools/os_win32.cpp?rev=f276b249fdc84019403491303479485030ac7c80#L1805

I'm trying to understand why it was put there. If you comment it out, everything works fine because ata_pass_through_ioctl is set up to handle multi-sector commands. I see your comment on the header of that method, but I don't think it applies anymore.

Can this assertion be removed to enable multi-sector data-out commands?

Change History (0)

Note: See TracTickets for help on using tickets.