Are you using this code for your own hardware?

Did you check if "fd" was valid? Looks like "fd" can be uninitialized.

Dave might be able to help you. I am not too familiar with the device
specific code.

-Ravi

On Jun 16, 3:18 am, Andy Quan <androidr...@gmail.com> wrote:
> Ravi,Could you help look at line 1045 of this file?
> Line 1045:
> sp<MemoryHeapBase> master = (MemoryHeapBase *) fd;
>
> fd is of uint32 and stands for file descriptor from OMX unit. But this line
> crashed as soon as it was reached. I guess this crash is because of
> definition of "sp".
>
> 314<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>template<typename
> T>
> 315<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>sp<T>&
> sp<T>::operator = (T* other)
> 316<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>{
> 317<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>
>    if (other) other->incStrong(this);
> 318<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>
>    if (m_ptr) m_ptr->decStrong(this);
> 319<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>
>    m_ptr = other;
> 320<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>
>    return *this;
> 321<http://android.git.kernel.org/?p=platform/frameworks/base.git;a=blob;...>}
>
> It seems that "fd->incStrong" is called but actually fd is only a file
> descriptor instead of a refbase object...
>
> Do you have any comment on this problem? Did I misunderstand anything? Thank
> you.
>
>
>
> On Tue, Jun 16, 2009 at 1:38 PM, Ravi <yend...@pv.com> wrote:
>
> > Look at the code in release-1.0.
>
> >http://android.git.kernel.org/?p=platform/external/opencore.git;a=blo...
>
> > This was definitely a working piece of code. A very close (if not the
> > same) version was used in the first device release. But the code has
> > changed quite a bit.
>
> > The code under "samples" is a version spun off from the original, and
> > is merely to demonstrate the usage.
>
> > -Ravi
>
> > On Jun 15, 11:57 pm, Andy Quan <androidr...@gmail.com> wrote:
> > > Hi,I find there are some sample files under
> > > external/opencore/android/samples demonstrating how child class of
> > > android_surface_output should be created. However, I did not find it
> > > eventually used in open source git. So my question is that, is this
> > sample
> > > file the same as the one used in G1 or HTC release?
> > > I reused this file but there come up some problems in PMEM usage. So I
> > > wonder if this is a verified source code or simply a demonstration. Thank
> > > you.
>
> > > --
> > > Thanks,
> > > Andy
>
> --
> Thanks,
> Andy
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"android-framework" group.
To post to this group, send email to android-framework@googlegroups.com
To unsubscribe from this group, send email to 
android-framework+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/android-framework?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to