Collin, can you take care of the comments and send out a new version?
On 10/20/2017 03:48 PM, Farhan Ali wrote: > > > On 10/20/2017 07:02 AM, Christian Borntraeger wrote: >> --- a/pc-bios/s390-ccw/sclp.c >> +++ b/pc-bios/s390-ccw/sclp.c >> @@ -83,7 +83,7 @@ long write(int fd, const void *str, size_t len) >> return -EIO; >> } >> >> - for (p = str; *p; ++p) { >> + for (p = str; len ; ++p, len--) { >> if (data_len > SCCB_DATA_LEN - 1) { >> return -EFBIG; >> } >> >> >> > The write function returns len, wouldn't this change make write return 0?