On (Wed) 20 Apr 2011 [13:30:13], Kevin Wolf wrote: > Signed-off-by: Kevin Wolf <kw...@redhat.com> > --- > hw/ide/atapi.c | 115 +++++++++++++++++++++++-------------------------------- > 1 files changed, 48 insertions(+), 67 deletions(-) > > diff --git a/hw/ide/atapi.c b/hw/ide/atapi.c > index d161bf7..d0bf7fd 100644 > --- a/hw/ide/atapi.c > +++ b/hw/ide/atapi.c > @@ -533,10 +533,11 @@ static unsigned int event_status_media(IDEState *s, > return 8; /* We wrote to 4 extra bytes from the header */ > } > > -static void handle_get_event_status_notification(IDEState *s, > - uint8_t *buf, > - const uint8_t *packet) > +static void cmd_get_event_status_notification(IDEState *s, > + uint8_t *buf) > { > + const uint8_t *packet = buf; > +
Exploiting packet == buf. But do we want to go away from that? Amit