Re: [PATCH] mtd: mtdram: check offs and len in mtdram->point madram->unpoint, mtdram->write and mtdram->read
Hi Chen, Can you use something shorter for your commit title. Something like "mtd: mtdram: check offs and len where appropriate" On Fri, 3 Mar 2017 15:31:00 +0800 chenwywrote: > We should prevent user to erasing mtd device with ^ from > an unaligned offset or length. Why are you putting 5 spaces before each line of your commit message? > > Signed-off-by: Chen Wenyong > --- > drivers/mtd/devices/mtdram.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/drivers/mtd/devices/mtdram.c b/drivers/mtd/devices/mtdram.c > index cbd8547..b0468c1 100644 > --- a/drivers/mtd/devices/mtdram.c > +++ b/drivers/mtd/devices/mtdram.c > @@ -67,6 +67,8 @@ static int ram_erase(struct mtd_info *mtd, struct > erase_info *instr) > static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, > size_t *retlen, void **virt, resource_size_t *phys) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Add an empty line. > *virt = mtd->priv + from; > *retlen = len; > return 0; > @@ -74,6 +76,8 @@ static int ram_point(struct mtd_info *mtd, loff_t from, > size_t len, > > static int ram_unpoint(struct mtd_info *mtd, loff_t from, size_t len) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Ditto. > return 0; > } > > @@ -93,6 +97,8 @@ static unsigned long ram_get_unmapped_area(struct mtd_info > *mtd, > static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, > size_t *retlen, u_char *buf) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Ditto. > memcpy(buf, mtd->priv + from, len); > *retlen = len; > return 0; > @@ -101,6 +107,8 @@ static int ram_read(struct mtd_info *mtd, loff_t from, > size_t len, > static int ram_write(struct mtd_info *mtd, loff_t to, size_t len, > size_t *retlen, const u_char *buf) > { > + if (check_offs_len(mtd, to, len)) > + return -EINVAL; Ditto. > memcpy((char *)mtd->priv + to, buf, len); > *retlen = len; > return 0;
Re: [PATCH] mtd: mtdram: check offs and len in mtdram->point madram->unpoint, mtdram->write and mtdram->read
Hi Chen, Can you use something shorter for your commit title. Something like "mtd: mtdram: check offs and len where appropriate" On Fri, 3 Mar 2017 15:31:00 +0800 chenwy wrote: > We should prevent user to erasing mtd device with ^ from > an unaligned offset or length. Why are you putting 5 spaces before each line of your commit message? > > Signed-off-by: Chen Wenyong > --- > drivers/mtd/devices/mtdram.c | 8 > 1 file changed, 8 insertions(+) > > diff --git a/drivers/mtd/devices/mtdram.c b/drivers/mtd/devices/mtdram.c > index cbd8547..b0468c1 100644 > --- a/drivers/mtd/devices/mtdram.c > +++ b/drivers/mtd/devices/mtdram.c > @@ -67,6 +67,8 @@ static int ram_erase(struct mtd_info *mtd, struct > erase_info *instr) > static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, > size_t *retlen, void **virt, resource_size_t *phys) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Add an empty line. > *virt = mtd->priv + from; > *retlen = len; > return 0; > @@ -74,6 +76,8 @@ static int ram_point(struct mtd_info *mtd, loff_t from, > size_t len, > > static int ram_unpoint(struct mtd_info *mtd, loff_t from, size_t len) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Ditto. > return 0; > } > > @@ -93,6 +97,8 @@ static unsigned long ram_get_unmapped_area(struct mtd_info > *mtd, > static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, > size_t *retlen, u_char *buf) > { > + if (check_offs_len(mtd, from, len)) > + return -EINVAL; Ditto. > memcpy(buf, mtd->priv + from, len); > *retlen = len; > return 0; > @@ -101,6 +107,8 @@ static int ram_read(struct mtd_info *mtd, loff_t from, > size_t len, > static int ram_write(struct mtd_info *mtd, loff_t to, size_t len, > size_t *retlen, const u_char *buf) > { > + if (check_offs_len(mtd, to, len)) > + return -EINVAL; Ditto. > memcpy((char *)mtd->priv + to, buf, len); > *retlen = len; > return 0;
[PATCH] mtd: mtdram: check offs and len in mtdram->point madram->unpoint,mtdram->write and mtdram->read
We should prevent user to erasing mtd device with an unaligned offset or length. Signed-off-by: Chen Wenyong--- drivers/mtd/devices/mtdram.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/mtd/devices/mtdram.c b/drivers/mtd/devices/mtdram.c index cbd8547..b0468c1 100644 --- a/drivers/mtd/devices/mtdram.c +++ b/drivers/mtd/devices/mtdram.c @@ -67,6 +67,8 @@ static int ram_erase(struct mtd_info *mtd, struct erase_info *instr) static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, size_t *retlen, void **virt, resource_size_t *phys) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; *virt = mtd->priv + from; *retlen = len; return 0; @@ -74,6 +76,8 @@ static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, static int ram_unpoint(struct mtd_info *mtd, loff_t from, size_t len) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; return 0; } @@ -93,6 +97,8 @@ static unsigned long ram_get_unmapped_area(struct mtd_info *mtd, static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, size_t *retlen, u_char *buf) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; memcpy(buf, mtd->priv + from, len); *retlen = len; return 0; @@ -101,6 +107,8 @@ static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, static int ram_write(struct mtd_info *mtd, loff_t to, size_t len, size_t *retlen, const u_char *buf) { + if (check_offs_len(mtd, to, len)) + return -EINVAL; memcpy((char *)mtd->priv + to, buf, len); *retlen = len; return 0; -- 2.9.3
[PATCH] mtd: mtdram: check offs and len in mtdram->point madram->unpoint,mtdram->write and mtdram->read
We should prevent user to erasing mtd device with an unaligned offset or length. Signed-off-by: Chen Wenyong --- drivers/mtd/devices/mtdram.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/mtd/devices/mtdram.c b/drivers/mtd/devices/mtdram.c index cbd8547..b0468c1 100644 --- a/drivers/mtd/devices/mtdram.c +++ b/drivers/mtd/devices/mtdram.c @@ -67,6 +67,8 @@ static int ram_erase(struct mtd_info *mtd, struct erase_info *instr) static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, size_t *retlen, void **virt, resource_size_t *phys) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; *virt = mtd->priv + from; *retlen = len; return 0; @@ -74,6 +76,8 @@ static int ram_point(struct mtd_info *mtd, loff_t from, size_t len, static int ram_unpoint(struct mtd_info *mtd, loff_t from, size_t len) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; return 0; } @@ -93,6 +97,8 @@ static unsigned long ram_get_unmapped_area(struct mtd_info *mtd, static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, size_t *retlen, u_char *buf) { + if (check_offs_len(mtd, from, len)) + return -EINVAL; memcpy(buf, mtd->priv + from, len); *retlen = len; return 0; @@ -101,6 +107,8 @@ static int ram_read(struct mtd_info *mtd, loff_t from, size_t len, static int ram_write(struct mtd_info *mtd, loff_t to, size_t len, size_t *retlen, const u_char *buf) { + if (check_offs_len(mtd, to, len)) + return -EINVAL; memcpy((char *)mtd->priv + to, buf, len); *retlen = len; return 0; -- 2.9.3