Message ID | 1499262995-11621-1-git-send-email-anoob.soman@citrix.com |
---|---|
State | New |
Headers | show |
On Wed, Jul 05, 2017 at 02:56:35PM +0100, Anoob Soman wrote: > xen_pt_pci_config_access_check checks if addr >= 0xFF. 0xFF is a valid > address and should not be ignored. > > Signed-off-by: Anoob Soman <anoob.soman@citrix.com> Acked-by: Anthony PERARD <anthony.perard@citrix.com> > --- > hw/xen/xen_pt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/xen/xen_pt.c b/hw/xen/xen_pt.c > index b6d71bb..375efa6 100644 > --- a/hw/xen/xen_pt.c > +++ b/hw/xen/xen_pt.c > @@ -85,7 +85,7 @@ void xen_pt_log(const PCIDevice *d, const char *f, ...) > static int xen_pt_pci_config_access_check(PCIDevice *d, uint32_t addr, int len) > { > /* check offset range */ > - if (addr >= 0xFF) { > + if (addr > 0xFF) { > XEN_PT_ERR(d, "Failed to access register with offset exceeding 0xFF. " > "(addr: 0x%02x, len: %d)\n", addr, len); > return -1;
diff --git a/hw/xen/xen_pt.c b/hw/xen/xen_pt.c index b6d71bb..375efa6 100644 --- a/hw/xen/xen_pt.c +++ b/hw/xen/xen_pt.c @@ -85,7 +85,7 @@ void xen_pt_log(const PCIDevice *d, const char *f, ...) static int xen_pt_pci_config_access_check(PCIDevice *d, uint32_t addr, int len) { /* check offset range */ - if (addr >= 0xFF) { + if (addr > 0xFF) { XEN_PT_ERR(d, "Failed to access register with offset exceeding 0xFF. " "(addr: 0x%02x, len: %d)\n", addr, len); return -1;
xen_pt_pci_config_access_check checks if addr >= 0xFF. 0xFF is a valid address and should not be ignored. Signed-off-by: Anoob Soman <anoob.soman@citrix.com> --- hw/xen/xen_pt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)