On Tue, 3 Nov 2009 15:09:54 -0500 Andres Salomon <[email protected]> wrote:
> On Mon, 2 Nov 2009 23:41:24 -0800 > Andrew Morton <[email protected]> wrote: > > > On Mon, 19 Oct 2009 01:37:08 -0400 Andres Salomon > > <[email protected]> wrote: > > > > > There are a few more macros in drmP.h that are unused; > > > DRM_GET_PRIV_SAREA, DRM_ARRAY_SIZE, and DRM_WAITCOUNT can go away > > > completely. > > > > > > Unfortunately, DRM_COPY is still used in one place, but we can at > > > least move it to where it's used. It's an awful looking macro.. > > > > It would have been nice to fix the (valid) checkpatch warnings while > > you were there. > > > > How about I do one better and replace the macro w/ a function? > > > > From 535091717408e4ec4974e8f309212f61aabd1880 Mon Sep 17 00:00:00 2001 > From: Andres Salomon <[email protected]> > Date: Tue, 3 Nov 2009 15:03:52 -0500 > Subject: [PATCH] drm: replace DRM_COPY macro w/ a function > > Don't inline it; the compiler can figure it out. Comments added that > are based upon my interpretation of the code. Hopefully they're > correct. :) > And here's one that actually checks the return values from drm_copy_field. This is separate from the pure cleanup patch(es) as it changes behavior that userspace sees, so it can be bisected and reverted apart from the others if needed. >From 041636a0e2b6d7ec5c4fdb8f495682d6928b73bf Mon Sep 17 00:00:00 2001 From: Andres Salomon <[email protected]> Date: Tue, 3 Nov 2009 15:15:51 -0500 Subject: [PATCH] drm: check return values in drm_version In drm_version, actually check the results from function calls so that we're not potentially passing garbage back to userspace. Signed-off-by: Andres Salomon <[email protected]> --- drivers/gpu/drm/drm_drv.c | 16 +++++++++++----- 1 files changed, 11 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c index ec0e3ae..5bd3f94 100644 --- a/drivers/gpu/drm/drm_drv.c +++ b/drivers/gpu/drm/drm_drv.c @@ -403,15 +403,21 @@ static int drm_version(struct drm_device *dev, void *data, struct drm_file *file_priv) { struct drm_version *version = data; + int err; version->version_major = dev->driver->major; version->version_minor = dev->driver->minor; version->version_patchlevel = dev->driver->patchlevel; - drm_copy_field(version->name, &version->name_len, dev->driver->name); - drm_copy_field(version->date, &version->date_len, dev->driver->date); - drm_copy_field(version->desc, &version->desc_len, dev->driver->desc); - - return 0; + err = drm_copy_field(version->name, &version->name_len, + dev->driver->name); + if (!err) + err = drm_copy_field(version->date, &version->date_len, + dev->driver->date); + if (!err) + err = drm_copy_field(version->desc, &version->desc_len, + dev->driver->desc); + + return err; } /** -- 1.5.6.5 ------------------------------------------------------------------------------ Come build with us! The BlackBerry(R) Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your developing skills, take BlackBerry mobile applications to market and stay ahead of the curve. Join us from November 9 - 12, 2009. Register now! http://p.sf.net/sfu/devconference -- _______________________________________________ Dri-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dri-devel
