maskit opened a new issue, #10352:
URL: https://github.com/apache/trafficserver/issues/10352

   cache_fill.cc
   ```
    69static bool
    70cont_check_cacheable(TSHttpTxn txnp)
    71{
        1. Condition tsapi::c::TSHttpTxnIsInternal(txnp), taking false branch.
    72  if (TSHttpTxnIsInternal(txnp)) {
    73    return false;
    74  }
    75  int lookupStatus;
        
   CID 1518583 (#1 of 1): Unchecked return value (CHECKED_RETURN)
   2. check_return: Calling TSHttpTxnCacheLookupStatusGet without checking 
return value (as is done elsewhere 17 out of 18 times).
    76  TSHttpTxnCacheLookupStatusGet(txnp, &lookupStatus);
    77  TSDebug(PLUGIN_NAME, "lookup status: %s", 
getCacheLookupResultName(static_cast<TSCacheLookupResult>(lookupStatus)));
    78  bool ret = false;
    79  if (TS_CACHE_LOOKUP_MISS == lookupStatus || TS_CACHE_LOOKUP_HIT_STALE 
== lookupStatus) {
    80    bool const nostore = TSHttpTxnServerRespNoStoreGet(txnp);
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@trafficserver.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to