diff options
author | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-11-05 13:06:35 +0100 |
---|---|---|
committer | Andreas Fritiofson <andreas.fritiofson@gmail.com> | 2011-11-14 23:39:20 +0000 |
commit | a943ec1ca9e8ef742ce0c0ae6b15acff960675ef (patch) | |
tree | 0816893d253bf0f8c48cc122b54779fbfa727939 /src/target | |
parent | 9e1a16690e669b895dce7c3951b1fe893bfd6149 (diff) | |
download | riscv-openocd-a943ec1ca9e8ef742ce0c0ae6b15acff960675ef.zip riscv-openocd-a943ec1ca9e8ef742ce0c0ae6b15acff960675ef.tar.gz riscv-openocd-a943ec1ca9e8ef742ce0c0ae6b15acff960675ef.tar.bz2 |
target: make it absolutely clear that no null pointers are accepted
there are no comments about what the rules w.r.t. null pointers
are and it is inconsistent.
It's simply a bug in the app if we ask about the properties of a null
pointer w.r.t. what kind of target it is. It's equally wrong to say
that it is an arm target as that it isn't an arm target.
Change-Id: I0925a6a5c8b38e594ffa7c3ca4390487b5e9b718
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Reviewed-on: http://openocd.zylin.com/168
Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Diffstat (limited to 'src/target')
-rw-r--r-- | src/target/arm.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/target/arm.h b/src/target/arm.h index ce8cbe1..63932e1 100644 --- a/src/target/arm.h +++ b/src/target/arm.h @@ -176,12 +176,14 @@ struct arm { /** Convert target handle to generic ARM target state handle. */ static inline struct arm *target_to_arm(struct target *target) { + assert(target != NULL); return target->arch_info; } static inline bool is_arm(struct arm *arm) { - return arm && arm->common_magic == ARM_COMMON_MAGIC; + assert(arm != NULL); + return arm->common_magic == ARM_COMMON_MAGIC; } struct arm_algorithm { |