Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build Instructions #1

Open
21212124 opened this issue Nov 18, 2019 · 9 comments
Open

Build Instructions #1

21212124 opened this issue Nov 18, 2019 · 9 comments

Comments

@21212124
Copy link

21212124 commented Nov 18, 2019

How do they apply the patches to the OP-TEE build system, client library and OS?
what is there to do?

@SilvaMatteus
Copy link

Hello @21212124.
When I was applying the patch on the build repository I got the following error:
error: corrupt patch at line 124
Have you experienced the same?

@21212124
Copy link
Author

21212124 commented Jan 8, 2020

I have no idea, but if you can get it started the fuzzer, let me know. thanks.

@21212124
Copy link
Author

@SilvaMatteus do you have many mistakes? It compiles me but I have warnings. When I run AFL I have the error: no instrumentation detected.
Have you experienced the same?

@SilvaMatteus
Copy link

SilvaMatteus commented Jan 22, 2020

Hi @21212124. It seems that you have the AFL compiled already. Could you share with me the steps you performed to compile?
I think it's better to share via e-mail.
matteus.sthefano.ls *AT* gmail.com

About your error: did you tried to run in dumb mode?

@21212124
Copy link
Author

Hi @SilvaMatteus
Any advance?

@SilvaMatteus
Copy link

Hi @21212124, I moved on to other priorities at the moment.

@SilvaMatteus
Copy link

Hi folks. I'm no longer working on this. Also, I do not have good news about these build instructions.

@RainyLave
Copy link

How do they apply the patches to the OP-TEE build system, client library and OS?
what is there to do?

hello,there,When I used the .diff files in the patches in the directory,I found that it aserted a lot of error,so I just patch the files by hand (not by my computer) on my self.And then I debuged it,It seems work well!

@bitwave
Copy link

bitwave commented Nov 11, 2021

I got them applied, by changing some line numbers :(

diff --git a/patches/optee-3.3/optee_client.diff b/patches/optee-3.3/optee_client.diff
index 60d1a70..a2a0c3e 100644
--- a/patches/optee-3.3/optee_client.diff
+++ b/patches/optee-3.3/optee_client.diff
@@ -53,7 +53,7 @@ index 698092b..0384e04 100644
  {
        struct tee_ioctl_close_session_arg arg;
  
-@@ -779,3 +787,183 @@ void TEEC_ReleaseSharedMemory(TEEC_SharedMemory *shm)
+@@ -779,3 +787,131 @@ void TEEC_ReleaseSharedMemory(TEEC_SharedMemory *shm)
        shm->registered_fd = -1;
        shm->buffer_allocated = false;
  }
@@ -185,4 +185,3 @@ index 698092b..0384e04 100644
 +
 +      _TEEC_CloseSession(session);
 +}
-\ No newline at end of file
diff --git a/patches/optee-3.3/optee_os.diff b/patches/optee-3.3/optee_os.diff
index dc54429..bbc5949 100644
--- a/patches/optee-3.3/optee_os.diff
+++ b/patches/optee-3.3/optee_os.diff
@@ -1916,7 +1916,7 @@ diff --git a/core/kernel/panic.c b/core/kernel/panic.c
 index 37e30a8..36af5e0 100644
 --- a/core/kernel/panic.c
 +++ b/core/kernel/panic.c
-@@ -8,11 +8,81 @@
+@@ -8,11 +8,14 @@
  #include <kernel/thread.h>
  #include <trace.h>
  

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants