Re: [PATCH] omap: pm: Fix error condition in _pwrdm_deps_lookup when pwrdm not found.
On Wed, 12 Aug 2009, Kevin Hilman wrote: > Mike Chan writes: > > > Check pwrdm_name instead of the address of a null struct when at the > > end of pwrdm_dep array. > > > > Reported-by: Paul Walmsley > > Signed-off-by: Mike Chan > > Thanks, to keep it all together, I'll revert the original and merge > this into a single patch. Great, then: Acked-by: Paul Walmsley > > Kevin > > > --- > > arch/arm/mach-omap2/powerdomain.c |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/arch/arm/mach-omap2/powerdomain.c > > b/arch/arm/mach-omap2/powerdomain.c > > index 0334609..02c1ef6 100644 > > --- a/arch/arm/mach-omap2/powerdomain.c > > +++ b/arch/arm/mach-omap2/powerdomain.c > > @@ -103,7 +103,7 @@ static struct powerdomain *_pwrdm_deps_lookup(struct > > powerdomain *pwrdm, > > > > } > > > > - if (!pd) > > + if (!pd->pwrdm_name) > > return ERR_PTR(-ENOENT); > > > > return pd->pwrdm; > > -- > > 1.5.4.5 > - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH] omap: pm: Fix error condition in _pwrdm_deps_lookup when pwrdm not found.
Mike Chan writes: > Check pwrdm_name instead of the address of a null struct when at the > end of pwrdm_dep array. > > Reported-by: Paul Walmsley > Signed-off-by: Mike Chan Thanks, to keep it all together, I'll revert the original and merge this into a single patch. Kevin > --- > arch/arm/mach-omap2/powerdomain.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-omap2/powerdomain.c > b/arch/arm/mach-omap2/powerdomain.c > index 0334609..02c1ef6 100644 > --- a/arch/arm/mach-omap2/powerdomain.c > +++ b/arch/arm/mach-omap2/powerdomain.c > @@ -103,7 +103,7 @@ static struct powerdomain *_pwrdm_deps_lookup(struct > powerdomain *pwrdm, > > } > > - if (!pd) > + if (!pd->pwrdm_name) > return ERR_PTR(-ENOENT); > > return pd->pwrdm; > -- > 1.5.4.5 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
[PATCH] omap: pm: Fix error condition in _pwrdm_deps_lookup when pwrdm not found.
Check pwrdm_name instead of the address of a null struct when at the end of pwrdm_dep array. Reported-by: Paul Walmsley Signed-off-by: Mike Chan --- arch/arm/mach-omap2/powerdomain.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-omap2/powerdomain.c b/arch/arm/mach-omap2/powerdomain.c index 0334609..02c1ef6 100644 --- a/arch/arm/mach-omap2/powerdomain.c +++ b/arch/arm/mach-omap2/powerdomain.c @@ -103,7 +103,7 @@ static struct powerdomain *_pwrdm_deps_lookup(struct powerdomain *pwrdm, } - if (!pd) + if (!pd->pwrdm_name) return ERR_PTR(-ENOENT); return pd->pwrdm; -- 1.5.4.5 -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html